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

[Skip CI] Change CV32E6?X to CV32E-X to ease file management #1705

Closed
wants to merge 4 commits into from
Closed

[Skip CI] Change CV32E6?X to CV32E-X to ease file management #1705

wants to merge 4 commits into from

Conversation

jquevremont
Copy link
Contributor

The placeholder CV32E6?X cannot be used in file names. The team decided to change the question mark ? to a dash -.

@jquevremont jquevremont added Component:Doc For issues in the Documentation (e.g. for README.md files) Type:Enhancement For feature requests and enhancements labels Dec 13, 2023
@JeanRochCoulon
Copy link
Contributor

I took the challenge to modify CVE-something by CVA-something. I will raise the topic in the Core task group. That's why I suggest to postpone this merge.

Copy link
Contributor

✔️ successful run, report available here.

@jquevremont jquevremont added the Status:Do-not-merge Pull request that should not be merged (yet) label Dec 14, 2023
@jquevremont
Copy link
Contributor Author

On hold until we get a consensus from the Cores TG.

@JeanRochCoulon
Copy link
Contributor

@jquevremont I propose to close this PR until the decision. The PR will be re-open when needed.

@jquevremont jquevremont deleted the jquevremont-remove_question_mark branch December 15, 2023 15:55
@jquevremont jquevremont restored the jquevremont-remove_question_mark branch December 15, 2023 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component:Doc For issues in the Documentation (e.g. for README.md files) Status:Do-not-merge Pull request that should not be merged (yet) Type:Enhancement For feature requests and enhancements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants