Skip to content

Conversation

@mosleyit
Copy link
Contributor

@mosleyit mosleyit commented Dec 6, 2024

Added:

'package-lock.json', 'yarn.lock', # Package lock files
'pnpm-lock.yaml', 'npm-shrinkwrap.json', # More package lock files
'LICENSE', 'LICENSE.', 'COPYING', # License files
'LICENCE', 'LICENCE.
', # Alternative spelling
'COPYING.', 'COPYRIGHT', # More license-related files
'AUTHORS', 'AUTHORS.
', # Author files
'CONTRIBUTORS', 'CONTRIBUTORS.', # Contributor files
'THANKS', 'THANKS.
', # Acknowledgment files
'CHANGELOG', 'CHANGELOG.', # Change logs
'CONTRIBUTING', 'CONTRIBUTING.
' # Contribution guidelines

@cyclotruc cyclotruc merged commit 61e34f9 into coderamp-labs:main Dec 6, 2024
@cyclotruc
Copy link
Member

Thank you very much

Good contribution, and good point for the separators!

@mosleyit
Copy link
Contributor Author

mosleyit commented Dec 6, 2024 via email

@cyclotruc
Copy link
Member

Great to hear!
Just so you know I'm currently splitting the files and restructuring a bit, so you might want to wait until I push to avoid many merge conflicts
I'll push asap (max 5 hours)

@cyclotruc
Copy link
Member

Most of the refactor is now done, we shouldn't expect too much structural change now and It's much easier to improve on this

FOLKS-Tech pushed a commit to FOLKS-Tech/gitingest that referenced this pull request Sep 5, 2025
* reduced other files to reduce clutter for the llm

* reduced the separator by 2 = this reduces that line 1 token for chatgpt does not affect anthropic claude
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants