Skip to content

Add --warn:reflective-naming (backport #2561)#2565

Merged
mergify[bot] merged 2 commits into3.5.xfrom
mergify/bp/3.5.x/pr-2561
Jun 6, 2022
Merged

Add --warn:reflective-naming (backport #2561)#2565
mergify[bot] merged 2 commits into3.5.xfrom
mergify/bp/3.5.x/pr-2561

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Jun 6, 2022

This is an automatic backport of pull request #2561 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

The new function is chisel3.internal.buildName.

(cherry picked from commit 370ca8a)
This new argument (and associated annotation) will turn on a warning
whenever reflective naming changes the name of a signal. This is
provided to help migrate from Chisel 3.5 to 3.6 since reflective naming
is removed in Chisel 3.6.

(cherry picked from commit 97afd9b)
@mergify mergify bot added the Backport Automated backport, please consider for minor release label Jun 6, 2022
@mergify mergify bot merged commit 42f5d89 into 3.5.x Jun 6, 2022
@mergify mergify bot deleted the mergify/bp/3.5.x/pr-2561 branch June 6, 2022 23:02
jackkoenig pushed a commit that referenced this pull request Feb 28, 2023
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backport Automated backport, please consider for minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant