-
Notifications
You must be signed in to change notification settings - Fork 89
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
docs: publish Dependabot TRG #659
Conversation
There is an open PR #60 related to dependabot.yml. The PR also addresses necessary description in the security_tooling.md file. |
Hi @tomaszbarwicki a bit unsure will this also create the new one into our default |
@FaGru3n well spotted! fixed, thanks :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The entry of dependabot.yml to trg-2-6 is not relevant. The updates related to dependabot are maintained under sig-security. Check 60.
Hi @RoKrish14 guess then we will need a separate TRG for Dependabot (how use it / and have it present for the release) here on our webpage that could link to your files within sig-security, but we have to make it clear that security is part of our TRG´s, but the hard requirements depend on your work. So guess @tomaszbarwicki wants to create this TRG and if you also want to spread the word for security tools maybe we can discuss the place like moving dependapot to the Security related TRG´s under the umcomming point 8. What do you think about that @tomaszbarwicki @RoKrish14 ? |
I agree to the need of "moving dependapot to the Security related TRG´s under the upcomming point 8" and @klaudiaZF is in the process of creating the TRG 8. |
Hi @RoKrish14, I'm open to move it to security section when it gets created. Shall we pause the PR or park it temporarily in the GIT section? |
@tomaszbarwicki : @klaudiaZF told me that your PR will go through with necessary changes. Once approved, it could be moved to TRG 8.0. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add npm and Gradle to yaml file and add information that open pull requests limits and interval can be set up based on team needs?
This wont be necessary. |
@tomaszbarwicki and @FaGru3n : "A basic example of a dependabot.yml file, demonstrating configurations for Docker and npm dependencies (all options), is shown below: " |
@klaudiaZF , @RoKrish14 incorporated your suggestions to add more dependabot config sample description informing about ability to customize certain parameters, included as well the link pointing to available ecosystem packages. Does that address your view? -> 666c60c |
Yes, thanks a lot. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks complete to me 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PR to publish Dependabot Tractus-X Release Guideline (TRG-2.06):