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.

Sign up for GitHub

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

Jump to bottom

GitHub actions Update #267

Merged
merged 29 commits into from
May 6, 2024
Merged

GitHub actions Update #267

merged 29 commits into from
May 6, 2024

Conversation

mikemcd3912
Copy link
Contributor

Issue #, if available:
N/A

Description of changes:

  1. Update Configmap Generation to ensure they are committed to the Partner folder for Tester and Partner addon file updates
  2. Update close behavior to back out all commits from GH action (Open and Sync) instead of just the latest commit
  3. Update close behavior to bring all accepted files over in the event of a merge for partner Addon updated and Internal PR's

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

mikemcd3912 and others added 29 commits May 1, 2024 20:52
@mikemcd3912
Actions Updates
2b524fb
@mikemcd3912
Comments on new function
2e406e1
@mikemcd3912
GH Actions work
265ebc5
@mikemcd3912
Revert "GH Actions work"
7554743 
This reverts commit 265ebc5.
@mikemcd3912
Set targets for development
d113e18
@mikemcd3912
Commit 1
31b1358
@mikemcd3912
Sync
e1a1ad0
@mikemcd3912
Update commit list iteration
bd1530e
@mikemcd3912
Merge pull request #33 from mikemcd3912/multi-commit
5c999ba 
Multi commit
@mikemcd3912
Revert "Multi commit"
d61e877
@mikemcd3912
Merge pull request #34 from mikemcd3912/revert-33-multi-commit
f774d80 
Revert "Multi commit"
@mikemcd3912
Add dependancy
9d73e40
@mikemcd3912
Commit 1
12f7d34
@mikemcd3912
Commit 2
f327e0b
@mikemcd3912
Merge pull request #35 from mikemcd3912/multi-commit-merge
294af94 
Commit merge
@mikemcd3912
Job naming correction
0b3fe67
@mikemcd3912
Removing Merged PR Files
39eb626
@mikemcd3912
Internal updates
cbe6ea6
@mikemcd3912
Fix paths ignore
790ed96
@mikemcd3912
Merge pull request #38 from mikemcd3912/internal-update-merge
ddf9a19 
Internal updates
@mikemcd3912
Remove merged test PR files
afc1799
@mikemcd3912
Testers Search update
81ab912
@mikemcd3912
new tester File
dcd8c58
@mikemcd3912
Sync
2b72b26
@mikemcd3912
Merge pull request #39 from mikemcd3912/partner-merge-sync-test
fe7d4b7 
new tester File
@mikemcd3912
Remove Merged PR39
d6333e3
@mikemcd3912
Delay adjustment from 120 to 60
dfb7f82
@mikemcd3912
Fix newrelic version variance
37a8167
@mikemcd3912
Re-direct branch target for merge
12dd7a8
@mikemcd3912 mikemcd3912 requested a review from elamaran11 May 3, 2024 18:59
elamaran11
elamaran11 approved these changes May 3, 2024
View reviewed changes
Copy link
Contributor

@elamaran11 elamaran11 left a comment

Choose a reason for hiding this comment

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

@mikemcd3912 Looks great.

@mikemcd3912 mikemcd3912 merged commit a85da31 into aws-samples:main May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@elamaran11 elamaran11 elamaran11 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mikemcd3912 @elamaran11