Skip to content

chore: add dependabot #40

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

Closed
wants to merge 1 commit into from

Conversation

itamm15
Copy link

@itamm15 itamm15 commented Jun 18, 2025

I am not a big fan of Dependabot (I prefer renovate), but it seems fine to have it there.

@itamm15 itamm15 force-pushed the chore/setup-dependabot branch from 0e807b7 to 41cfa36 Compare June 18, 2025 09:42
@itamm15 itamm15 force-pushed the chore/setup-dependabot branch from 41cfa36 to b79e4a4 Compare June 18, 2025 09:42
@patatoid
Copy link
Collaborator

There is no mix.lock file within the package, will this still work?

@itamm15
Copy link
Author

itamm15 commented Jun 18, 2025

Hey @patatoid! Do you mean within the dependabot.yaml? If so, then yes, that should work.
I just noticed that there is no mix.lock file in the repo (it is ignored via .gitignore). Is there any particular reason for that? Without `mix. lock ' file, the dependabot might not work correctly for minor updates, afair.

@patatoid
Copy link
Collaborator

patatoid commented Jun 18, 2025

@itamm15 there is no mix.lock because the library is to be integrated to existing projects. It helps to avoid conflicts with them.

Only major versions of the dependencies require an upgrade to be up to date. I am not sure dependabot is of any help here. If there are any, the best is to open a pull request depending on the needs.

@itamm15
Copy link
Author

itamm15 commented Jun 18, 2025

Hmm, I am not sure I get it? 🤔 Looking at other libs, like credo, ecto or gettext, they all have the mix.lock file.

@patatoid
Copy link
Collaborator

They may have no running applications, having a mix.lock file conflicts with the main application in my remembering.

@patatoid patatoid closed this Jun 30, 2025
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