Skip to content

Comments

Closes #1787 - Remove '-review' from reviewDomain()#1788

Closed
Levistator wants to merge 1 commit intogitkraken:mainfrom
Levistator:bugfix/gerrit_remote
Closed

Closes #1787 - Remove '-review' from reviewDomain()#1788
Levistator wants to merge 1 commit intogitkraken:mainfrom
Levistator:bugfix/gerrit_remote

Conversation

@Levistator
Copy link


Description

Checklist

  • I have followed the guidelines in the Contributing document
  • My changes follow the coding style of this project
  • My changes build without any errors or warnings
  • My changes have been formatted and linted
  • My changes include any required corresponding changes to the documentation
  • My changes have been rebased and squashed to the minimal number (typically 1) of relevant commits
  • My changes have a descriptive commit message with a short title, including a Fixes $XXX - or Closes #XXX - prefix to auto-close the issue that your PR addresses

Was breaking integration with my installation of gerrit.
Why should the subdomain be assumed to be anything other than what was
directly defined in the settings.json
@felipecrs
Copy link
Contributor

I believe #1954 also fixes your problem.

@felipecrs felipecrs mentioned this pull request Apr 7, 2022
7 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