-
Notifications
You must be signed in to change notification settings - Fork 14
Autogenerate docs #159
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
Autogenerate docs #159
Conversation
Close #164 |
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.
Good job, some minor changes and we are good
repository: 'outscale-vbr/image-builder' | ||
path: "add-outscale-image" | ||
ref: add-outscale-image |
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.
Any reason to go back to our fork ?
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.
@outscale-mdr
https://github.com/outscale-vbr/image-builder/pull/2/files
Yes because packer use now outscale instead of osc.
I have to add a new PR to image builder.
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.
When those PR (and also the one with global permission ) will be accept, review, merged, we will switch back to the official repo.
@outscale-mdr @outscale-hmi Are you ok with the latest changes ? |
GH_ORG_NAME ?= outscale-dev | ||
GH_REPO_NAME ?= cluster-api-provider-outscale | ||
GIT_USERNAME ?= Outscale Bot | ||
GIT_USEREMAIL ?= [email protected] |
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.
secret or not ? :)
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.
It is on both secret to override it
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.
Remove from secret
@outscale-hmi Are you ok with latest changes ? |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Auto generate docs version.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #160
Special notes for your reviewer:
Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.
TODOs: