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

Test: Rename: ChargebackRateDetailCurrency -> Currency #19

Conversation

lpichler
Copy link
Contributor

@lpichler lpichler commented Nov 21, 2019

@lpichler lpichler force-pushed the test_rename_ChargebackRateDetailCurrencyToCurrency branch 3 times, most recently from 65ba47d to e49701a Compare November 21, 2019 14:18
@lpichler
Copy link
Contributor Author

ManageIQ/manageiq#19542

@lpichler lpichler force-pushed the test_rename_ChargebackRateDetailCurrencyToCurrency branch 3 times, most recently from 585e10e to 69174ae Compare November 28, 2019 12:05
.travis.yml Outdated
@@ -15,5 +15,31 @@ matrix:
env:
global:
- REPOS=
- CORE_REPO=manageiq#19538
Copy link

@himdel himdel Nov 28, 2019

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

A PR should be # a commit/branch/tag should be @
Looks like the readme is wrong, we follow the standard GitHub short format.

Copy link
Member

Choose a reason for hiding this comment

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

Also CORE_REPO vs GEM_REPOS are no longer necessary, they're combined into just REPOS

@lpichler lpichler force-pushed the test_rename_ChargebackRateDetailCurrencyToCurrency branch 3 times, most recently from bea678b to 7eac9ba Compare December 4, 2019 19:01
@lpichler lpichler force-pushed the test_rename_ChargebackRateDetailCurrencyToCurrency branch from 7eac9ba to ad4aa50 Compare December 5, 2019 09:19
@lpichler lpichler closed this Dec 9, 2019
@lpichler lpichler reopened this Dec 9, 2019
@lpichler lpichler closed this Dec 9, 2019
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.

3 participants