Skip to content

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

Merged
2 commits merged into from Nov 16, 2022
Merged

Autogenerate docs #159

2 commits merged into from Nov 16, 2022

Conversation

ghost
Copy link

@ghost ghost commented Oct 18, 2022

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:

  • squashed commits
  • includes documentation
  • adds unit tests

@ghost ghost requested review from outscale-hmi and removed request for outscale-hmi October 18, 2022 17:04
@ghost ghost assigned outscale-hmi and unassigned outscale-hmi Oct 21, 2022
@ghost ghost requested a review from outscale-hmi October 21, 2022 09:11
@ghost
Copy link
Author

ghost commented Oct 26, 2022

Close #164

@ghost ghost requested a review from outscale-mdr October 31, 2022 13:27
Copy link
Contributor

@outscale-mdr outscale-mdr left a 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

Comment on lines +13 to +15
repository: 'outscale-vbr/image-builder'
path: "add-outscale-image"
ref: add-outscale-image
Copy link
Contributor

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 ?

Copy link
Author

@ghost ghost Nov 3, 2022

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.

Copy link
Author

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.

@ghost ghost requested a review from outscale-mdr November 3, 2022 09:20
@ghost
Copy link
Author

ghost commented Nov 7, 2022

@outscale-mdr @outscale-hmi Are you ok with the latest changes ?

Comment on lines +28 to +31
GH_ORG_NAME ?= outscale-dev
GH_REPO_NAME ?= cluster-api-provider-outscale
GIT_USERNAME ?= Outscale Bot
GIT_USEREMAIL ?= [email protected]
Copy link
Contributor

Choose a reason for hiding this comment

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

secret or not ? :)

Copy link
Author

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

Copy link
Author

Choose a reason for hiding this comment

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

Remove from secret

@ghost
Copy link
Author

ghost commented Nov 10, 2022

@outscale-hmi Are you ok with latest changes ?

@ghost ghost merged commit 6ac132a into outscale:main Nov 16, 2022
@ghost ghost added the kind/feature Feature resolution label Nov 25, 2022
@ghost ghost added this to the v0.1.2 milestone Nov 25, 2022
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Feature resolution
Development

Successfully merging this pull request may close these issues.

Autogenerate doc with image name
3 participants