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

Update pull request template with working links #156

Merged
merged 1 commit into from
Aug 15, 2023

Conversation

xibz
Copy link
Contributor

@xibz xibz commented Aug 14, 2023

Changes

A couple links seems to be outdated which results in 404s. This commit addresses that by pointing to the CDEvents primer website with the appropriate sub-links.

Submitter Checklist

As the author of this PR, please check off the items in this checklist:

.github/pull_request_template.md Outdated Show resolved Hide resolved
.github/pull_request_template.md Outdated Show resolved Hide resolved
@e-backmark-ericsson
Copy link
Contributor

Almost there, but the commit doesn't seem to be signed. We have a requirement on that all commits merged in cdevents shall be signed.
https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits

A couple links seems to be outdated which results in 404s. This commit
addresses that by pointing to the CDEvents primer website with the
appropriate sub-links.

Co-authored-by: Emil Bäckmark <emil.backmark@ericsson.com>
Signed-off-by: benjamin-j-powell <benjamin_j_powell@apple.com>
@xibz
Copy link
Contributor Author

xibz commented Aug 15, 2023

@e-backmark-ericsson done!

@e-backmark-ericsson e-backmark-ericsson merged commit d5fbf11 into cdevents:main Aug 15, 2023
3 checks passed
xibz added a commit to xibz/spec that referenced this pull request Oct 13, 2023
A couple links seems to be outdated which results in 404s. This commit
addresses that by pointing to the CDEvents primer website with the
appropriate sub-links.

Signed-off-by: benjamin-j-powell <benjamin_j_powell@apple.com>
Co-authored-by: benjamin-j-powell <benjamin_j_powell@apple.com>
Co-authored-by: Emil Bäckmark <emil.backmark@ericsson.com>
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