-
Notifications
You must be signed in to change notification settings - Fork 325
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
PE-37978: Add 'amazon' to #repo-filename method #1858
PE-37978: Add 'amazon' to #repo-filename method #1858
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1858 +/- ##
=======================================
Coverage 74.53% 74.53%
=======================================
Files 81 81
Lines 4504 4504
=======================================
Hits 3357 3357
Misses 1147 1147 ☔ View full report in Codecov by Sentry. |
b8b92d6
to
057a7f2
Compare
@span786 can you please rebase so we've a single commit in this PR? |
Revert "[PE-37978]: Add 'amazon' to repo_filename method with tests" This reverts commit 47f1238. [PE-37978]: Add 'amazon' to repo_filename method with tests
057a7f2
to
c8e03a2
Compare
Hi @bastelfreak, I have squashed the commits into 1. |
@@ -133,6 +133,14 @@ def to_s | |||
expect(filename).to be === correct | |||
end | |||
|
|||
it 'builds the filename correctly for amazon-based platforms' do | |||
@platform = 'amazon-2023-x86_64' |
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 this be made date independant?
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.
No, the 2023 is the release version. Sort of like how Ubuntu has it.
Added 'amazon' support in #repo_filename method in lib/beaker/host/unix/file.rb.