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

Extract IIAB_REMOTE_URL from 1 of 2 places, for both Git repos {iiab, iiab-admin-console} and... #3291

Merged
merged 2 commits into from
Jul 6, 2022

Conversation

@holta holta added this to the 8.0 milestone Jul 6, 2022
@holta
Copy link
Member Author

holta commented Jul 6, 2022

FYI this PR brings together a lot, hopefully in a concise package allowing iiab-summary to be very useful to many more QA People and others (it's also included at the top of iiab-diagnostics now).

So it's possible (maybe?) that git log HEAD^2 -1 --oneline (commit + branch name of PR being tested) might not be needed in the end ?

As these details are now prominently included in /etc/iiab/pr-list-pulled

In any case let's keep an eye on it. And finally if necessary, refactoring can later happen along these lines or similarly:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant