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

CLOUDP-240677 Update SDK update process to handle minor versions and create jiras #2825

Merged
merged 13 commits into from
Apr 5, 2024

Conversation

fmenezes
Copy link
Collaborator

@fmenezes fmenezes commented Apr 5, 2024

Proposed changes

Update SDK update process to handle minor versions and create jiras
Jira ticket: CLOUDP-240677

Closes #[issue number]

Checklist

  • I have signed the MongoDB CLA
  • I have added tests that prove my fix is effective or that my feature works
  • I have added any necessary documentation in document requirements section listed in CONTRIBUTING.md (if appropriate)
  • I have addressed the @mongodb/docs-cloud-team comments (if appropriate)
  • I have updated test/README.md (if an e2e test has been added)
  • I have run make fmt and formatted my code

Further comments

@fmenezes fmenezes changed the title CLOUDP 240677 CLOUDP-240677 Update SDK update process to handle minor versions and create jiras Apr 5, 2024
@fmenezes fmenezes marked this pull request as ready for review April 5, 2024 10:29
@fmenezes fmenezes requested a review from a team April 5, 2024 10:29
@fmenezes
Copy link
Collaborator Author

fmenezes commented Apr 5, 2024

this works and verified, created #2826 and https://jira.mongodb.org/browse/CLOUDP-241394

- name: Verify Changed files
uses: tj-actions/verify-changed-files@1e517a7f5663673148ceb7c09c1900e5af48e7a1
id: verify-changed-files
with:
files: |
./internal/**/*
go.mod
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

this is the actual change to make it work for minor and patch versions

@@ -8,6 +8,8 @@ updates:
open-pull-requests-limit: 20
reviewers:
- "mongodb/mongocli"
ignore:
- dependency-name: "go.mongodb.org/atlas-sdk/*"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ignoring on dependabot

Copy link
Member

Choose a reason for hiding this comment

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

Actualy I'm not sure that we should do this. Do we have other job covering minor updates?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

this one

wtrocki
wtrocki previously approved these changes Apr 5, 2024
Copy link
Member

@wtrocki wtrocki left a comment

Choose a reason for hiding this comment

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

LGTM

@fmenezes fmenezes merged commit c578187 into master Apr 5, 2024
17 checks passed
@fmenezes fmenezes deleted the CLOUDP-240677 branch April 5, 2024 11:59
@fmenezes fmenezes mentioned this pull request Apr 5, 2024
6 tasks
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