Skip to content

Conversation

@jbampton
Copy link
Member

Did you read the Contributor Guide?

Is this PR related to a JIRA ticket?

  • No:
    • this is a CI update. The PR name follows the format [CI] my subject

What changes were proposed in this PR?

Let us automate all the things :)

Some other PRs like this one have already been merged.

How was this patch tested?

Ran pre-commit run --all-files both before and after the license headers were added.

After the license headers were added to the Markdown files I ran:

mkdocs serve

and browsed the generated site locally doing a visual inspection of some of the web pages.

Did this PR include necessary documentation updates?

  • No, this PR does not affect any public API so no need to change the documentation.

@jbampton jbampton force-pushed the auto-add-license-for-markdown branch from 7462a91 to 6b28ed3 Compare November 25, 2024 18:26
@jbampton jbampton marked this pull request as ready for review November 25, 2024 19:19
@jiayuasu
Copy link
Member

jiayuasu commented Dec 3, 2024

Can you address the conflict? @jbampton

@jiayuasu jiayuasu merged commit 2331611 into apache:master Dec 4, 2024
34 checks passed
@jbampton jbampton deleted the auto-add-license-for-markdown branch December 9, 2024 21:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants