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

docs: add Pull Request Template #2784

Merged
merged 1 commit into from
Oct 18, 2023

Conversation

ruzell22
Copy link
Contributor

@ruzell22 ruzell22 commented Oct 12, 2023

Checkbox is updated so it will reflect as expected.

fixes: #91
related to: #2708

Signed-off-by: ruzell22 [email protected]

Copy link
Contributor

@jagpreetsinghsasan jagpreetsinghsasan left a comment

Choose a reason for hiding this comment

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

The code isnt rendered correctly: https://github.com/ruzell22/cactus/blob/prtemplate91/PULL_REQUEST_TEMPLATE.md (No new lines)

You can install Markdown All in One extension in vscode and then press Ctrl + Shift + V to check the rendered output

Copy link
Contributor

@petermetz petermetz left a comment

Choose a reason for hiding this comment

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

^^ +1

@ruzell22 ruzell22 force-pushed the prtemplate91 branch 2 times, most recently from bc9ae99 to 2c2aad1 Compare October 13, 2023 04:56
@ruzell22
Copy link
Contributor Author

Hello @jagpreetsinghsasan @petermetz , pushed a changed and the md file is now displaying the checkboxes as expected.
https://github.com/ruzell22/cactus/blob/prtemplate91/PULL_REQUEST_TEMPLATE.md
image

Copy link
Contributor

@jagpreetsinghsasan jagpreetsinghsasan left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@petermetz petermetz left a comment

Choose a reason for hiding this comment

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

@ruzell22 LGTM - but please add a [skip ci] declaration to the commit message so that we are not burning money on the CI for docs (the check that cancels the CI run if it's docs only is still not working so we have to try and pay some attention to this manually unfortunately.

@ruzell22
Copy link
Contributor Author

@petermetz noted on that Peter, thank you. I have updated the commit message with [skip ci] .

@petermetz petermetz enabled auto-merge (rebase) October 13, 2023 18:30
Checkbox is updated so it will reflect as expected.

fixes: hyperledger-cacti#91
related to: hyperledger-cacti#2708

Signed-off-by: ruzell22 <[email protected]>

[skip ci]
@petermetz petermetz merged commit 1cc3fcc into hyperledger-cacti:main Oct 18, 2023
4 checks passed
@petermetz petermetz deleted the prtemplate91 branch October 18, 2023 06:00
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.

docs: add Pull Request Template
4 participants