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

DEVPROD-12194 update PR description #8473

Merged
merged 3 commits into from
Nov 25, 2024
Merged

Conversation

ablack12
Copy link
Contributor

@ablack12 ablack12 commented Nov 11, 2024

DEVPROD-12194

Description

Make a note to test pine for markdown only changes that could break it.

@ablack12 ablack12 requested a review from a team November 11, 2024 17:13
Comment on lines 10 to 12
Remember to add or edit docs in the docs/ directory if relevant.
<!-- If you're only updating docs, create a patch for the Pine tasks to ensure our changes are compatible,
if you're making any structural changes (for example, adding links or new pages)-->
Copy link
Member

Choose a reason for hiding this comment

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

I understand
"If you're only updating docs, create a patch for the Pine tasks to ensure our changes are compatible"
but I don't understand this part:
"if you're making any structural changes (for example, adding links or new pages)"
What should be done in this case?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah i meant you only need to create a patch if you're making actual structural changes. this is poorly worded ,let me tweak

@ablack12 ablack12 requested a review from malikchaya2 November 22, 2024 18:51
@ablack12 ablack12 merged commit 8ff0291 into evergreen-ci:main Nov 25, 2024
5 checks passed
@ablack12 ablack12 deleted the DEVPROD-12194 branch November 25, 2024 18:50
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