Skip to content

Conversation

@michaelsembwever
Copy link
Member

https://cwiki.apache.org/confluence/display/INFRA/Git+-+.asf.yaml+features
And remove a few dangling copyright headers that were missed in 3923141

patch by Mick Semb Wever; reviewed by xxx for CASSANALYTICS-15

Copy link

@JeetKunDoug JeetKunDoug left a comment

Choose a reason for hiding this comment

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

+1 - thanks for adding the .asf.yml and the copyright notice cleanup.

 https://cwiki.apache.org/confluence/display/INFRA/Git+-+.asf.yaml+features
And remove a few dangling copyright headers that were missed in 3923141

 patch by Mick Semb Wever; reviewed by Doug Rohrer for CASSANALYTICS-15
@michaelsembwever michaelsembwever force-pushed the mck/CASSANALYTICS-44/trunk branch from a8ed2d8 to 27e6989 Compare April 22, 2025 18:17
@michaelsembwever
Copy link
Member Author

added another commit to fix some urls and trunk branch name. also to get GHA working again for PRs against trunk

@michaelsembwever michaelsembwever force-pushed the mck/CASSANALYTICS-44/trunk branch from 18de4c1 to ab16cc5 Compare April 23, 2025 18:54
@michaelsembwever
Copy link
Member Author

and another commit to get the GHA working…

@michaelsembwever
Copy link
Member Author

@JeetKunDoug , need to ask you to quickly review the two small additional commits here 🙏

Copy link

@JeetKunDoug JeetKunDoug left a comment

Choose a reason for hiding this comment

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

A few comments (one that doesn't have anything to do with the review itself, more a policy/procedure question for discussion maybe on Slack) but looking good.

@michaelsembwever michaelsembwever force-pushed the mck/CASSANALYTICS-44/trunk branch from ab16cc5 to f90322c Compare April 24, 2025 17:22
@michaelsembwever michaelsembwever deleted the mck/CASSANALYTICS-44/trunk branch April 28, 2025 12:04
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