Skip to content

Add dependabot #282

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

Merged
merged 1 commit into from
Aug 30, 2024
Merged

Add dependabot #282

merged 1 commit into from
Aug 30, 2024

Conversation

capital-G
Copy link
Contributor

Makes it easier to keep the dependencies of the website up to date and be aware when something breaks.

@capital-G
Copy link
Contributor Author

@dyfer should we merge this? Seems you were fond of it regarding sc3-plugins^^

@dyfer
Copy link
Member

dyfer commented Aug 27, 2024

Hm, I'd give it some time to see how it plays out on sc3-plugins. As I don't really follow github pages and jekyll environments, I don't know what the possible side effects could be. What do you think about that?

In particular, there's still the warning from pages build and deployment action (The github-pages gem can't satisfy your Gemfile's dependencies.). I'm guessing that dependabot is not meant to solve this, but the fact that we'd have dependencies needed by jekyll (or whatever it is that we're using) and possibly security updates coming in from the dependabot action makes me a bit uneasy. Mostly because I have not "big picture" understanding of the github pages system and I don't have mental space to really dive into it.

Thoughts?

@capital-G
Copy link
Contributor Author

I also don't know much about jekyll/ruby etc. and its dependencies - on the other hand, as long as the website builds it is fine? We have a CI step which builds the website, so as long as this works any upgardes suggested by dependabot could be "checked" this way.

@dyfer
Copy link
Member

dyfer commented Aug 30, 2024

Sounds good, let's try this! We can always remove it if it doesn't work as desired.

@dyfer dyfer closed this Aug 30, 2024
@dyfer dyfer reopened this Aug 30, 2024
@dyfer dyfer merged commit 4819e3d into supercollider:main Aug 30, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants