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

Use registry.redhat.io to load server and operator #20

Merged
merged 6 commits into from
Nov 22, 2019

Conversation

dmytro-ndp
Copy link
Contributor

@dmytro-ndp dmytro-ndp commented Nov 21, 2019

Fix default operator and server addresses to use registry.redhat.io instead of quay.io by default.

Issue to be fixed

https://issues.jboss.org/browse/CRW-506

Fix default operator and server addresses to use registry.redhat.io by default.
Signed-off-by: Dmytro Nochevnov <dnochevn@redhat.com>
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Copy link
Member

@ibuziuk ibuziuk left a comment

Choose a reason for hiding this comment

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

@dmytro-ndp AFAIK, README is updated automatically by npx, so need to update it manually cc: @benoitf

Signed-off-by: Dmytro Nochevnov <dnochevn@redhat.com>
@dmytro-ndp
Copy link
Contributor Author

dmytro-ndp commented Nov 21, 2019

@ibuziuk: in that case it's strange that there are other commits regarding README.md, for example a12cc1f#diff-04c6e90faac2675aa89e2176d2eec7d8

@ibuziuk
Copy link
Member

ibuziuk commented Nov 21, 2019

@dmytro-ndp I mean someone need to run the command for update and than commit changes, but not fix the README manually. @benoitf could you please comment how this should be done?

Signed-off-by: Dmytro Nochevnov <dnochevn@redhat.com>
Signed-off-by: Dmytro Nochevnov <dnochevn@redhat.com>
@dmytro-ndp
Copy link
Contributor Author

dmytro-ndp commented Nov 22, 2019

Meanwhile I have restored README.md to unblock fixup.

Copy link
Collaborator

@benoitf benoitf left a comment

Choose a reason for hiding this comment

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

I've added through PR an update of README.MD (using npx oclif-dev pack)

@dmytro-ndp
Copy link
Contributor Author

Thank you, @benoitf !

@dmytro-ndp dmytro-ndp merged commit 4edf39d into master Nov 22, 2019
@dmytro-ndp dmytro-ndp deleted the dmytro-ndp-patch-2 branch November 22, 2019 12:45
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.

None yet

4 participants