Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

git integration #141

Merged
merged 13 commits into from
Jan 4, 2024
Merged

git integration #141

merged 13 commits into from
Jan 4, 2024

Conversation

c0rydoras
Copy link
Collaborator

@c0rydoras c0rydoras commented Oct 31, 2023

Resolves #43

Possible improvements:

  • parser
    • use poetry directly to parse and interpret lockfiles instead of tomllib
    • use pyarn for yarn.lock instead of regex as its tested
    • add yarn@^3.0.0 parser, this should be easy as modern yarn.lock files are yaml
    • add support for yarn.lock past yarn classic. this does not require an extra library as its yaml
    • use dependencies and devDependencies keys for pnpm-lock.yaml instead of packages and regex

@c0rydoras c0rydoras added backend This issue or pull request is backend related feature This issue or pull request discusses a feature breaking This issue or pull request discusses something breaking labels Oct 31, 2023
@c0rydoras c0rydoras added this to the Outdated v1 milestone Oct 31, 2023
@c0rydoras c0rydoras self-assigned this Oct 31, 2023
@c0rydoras c0rydoras force-pushed the git-integration branch 2 times, most recently from 289bb17 to 7f6fc97 Compare November 1, 2023 07:36
@c0rydoras c0rydoras removed this from the Outdated v1 milestone Nov 1, 2023
@c0rydoras c0rydoras force-pushed the git-integration branch 2 times, most recently from bf12dec to 345777f Compare November 3, 2023 12:29
@c0rydoras c0rydoras force-pushed the git-integration branch 3 times, most recently from e22d3ee to c937d1b Compare November 6, 2023 10:05
Copy link
Member

@open-dynaMIX open-dynaMIX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work 🚀

api/outdated/models.py Outdated Show resolved Hide resolved
api/outdated/tests/test_repo_field.py Outdated Show resolved Hide resolved
api/outdated/outdated/tracking.py Show resolved Hide resolved
api/outdated/conftest.py Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
@c0rydoras c0rydoras merged commit 8225a40 into adfinis:main Jan 4, 2024
4 checks passed
@c0rydoras c0rydoras deleted the git-integration branch January 4, 2024 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend This issue or pull request is backend related breaking This issue or pull request discusses something breaking feature This issue or pull request discusses a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use git to crawl for dependencies
2 participants