Skip to content

Replaced references to master with references to main in docs/maintainers_guide.rst#11197

Merged
jakelishman merged 1 commit into
Qiskit:mainfrom
matthewscottconroy:change-master-to-main-in-docs-maintainers-page
Nov 6, 2023
Merged

Replaced references to master with references to main in docs/maintainers_guide.rst#11197
jakelishman merged 1 commit into
Qiskit:mainfrom
matthewscottconroy:change-master-to-main-in-docs-maintainers-page

Conversation

@matthewscottconroy
Copy link
Copy Markdown
Contributor

Replaced references to master with references to main in docs/maintainers_guide.rst (#11196)

  • Added a release notes file

  • Substituted mentions of the master branch with mentions to main in the file docs/maintainers_guide.rst

Summary

fixes #11196 (occurrences of master are replaced with occurrences of main)

Details and comments

There is currently only a main branch, so mentions of master seem to intend to refer to the main branch. The text in docs/maintainers_guide.rst has been adjusted accordingly so that all mentions of master are now mentions of main.

@matthewscottconroy matthewscottconroy requested a review from a team as a code owner November 6, 2023 05:03
@qiskit-bot qiskit-bot added the Community PR PRs from contributors that are not 'members' of the Qiskit repo label Nov 6, 2023
@qiskit-bot
Copy link
Copy Markdown
Collaborator

Thank you for opening a new pull request.

Before your PR can be merged it will first need to pass continuous integration tests and be reviewed. Sometimes the review process can be slow, so please be patient.

While you're waiting, please feel free to review other open PRs. While only a subset of people are authorized to approve pull requests for merging, everyone is encouraged to review open pull requests. Doing reviews helps reduce the burden on the core team and helps make the project's code better for everyone.

One or more of the the following people are requested to review this:

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Nov 6, 2023

CLA assistant check
All committers have signed the CLA.

Copy link
Copy Markdown
Member

@jakelishman jakelishman left a comment

Choose a reason for hiding this comment

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

Nice spot, and thanks for the fix!

We don't actually put a release note for documentation-only changes - would you be able to remove it from the PR? Other than that, this is good to merge, thank you!

@matthewscottconroy matthewscottconroy force-pushed the change-master-to-main-in-docs-maintainers-page branch from ec5615e to 14728cf Compare November 6, 2023 14:27
@matthewscottconroy matthewscottconroy force-pushed the change-master-to-main-in-docs-maintainers-page branch from 14728cf to 14f9917 Compare November 6, 2023 14:34
…ners_guide.rst (Qiskit#11196)

* Substituted mentions of the master branch with mentions to main in the
  file docs/maintainers_guide.rst
@matthewscottconroy matthewscottconroy force-pushed the change-master-to-main-in-docs-maintainers-page branch from 14f9917 to 22fd36b Compare November 6, 2023 14:53
Copy link
Copy Markdown
Member

@jakelishman jakelishman left a comment

Choose a reason for hiding this comment

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

Ace, thanks!

@jakelishman jakelishman enabled auto-merge November 6, 2023 14:54
@jakelishman jakelishman added documentation Something is not clear or an error documentation stable backport potential Make Mergify open a backport PR to the most recent stable branch on merge. Changelog: None Do not include in the GitHub Release changelog. labels Nov 6, 2023
@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 6772459862

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 11 unchanged lines in 3 files lost coverage.
  • Overall coverage increased (+0.009%) to 86.942%

Files with Coverage Reduction New Missed Lines %
crates/qasm2/src/lex.rs 2 91.41%
qiskit/pulse/library/waveform.py 3 93.75%
crates/qasm2/src/parse.rs 6 97.6%
Totals Coverage Status
Change from base Build 6772031961: 0.009%
Covered Lines: 74355
Relevant Lines: 85523

💛 - Coveralls

@jakelishman jakelishman added this pull request to the merge queue Nov 6, 2023
Merged via the queue into Qiskit:main with commit af643eb Nov 6, 2023
mergify Bot pushed a commit that referenced this pull request Nov 6, 2023
…ners_guide.rst (#11196) (#11197)

* Substituted mentions of the master branch with mentions to main in the
  file docs/maintainers_guide.rst

(cherry picked from commit af643eb)
github-merge-queue Bot pushed a commit that referenced this pull request Nov 6, 2023
…ners_guide.rst (#11196) (#11197) (#11203)

* Substituted mentions of the master branch with mentions to main in the
  file docs/maintainers_guide.rst

(cherry picked from commit af643eb)

Co-authored-by: Matthew Scott Conroy <matthewscottconroy@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changelog: None Do not include in the GitHub Release changelog. Community PR PRs from contributors that are not 'members' of the Qiskit repo documentation Something is not clear or an error documentation stable backport potential Make Mergify open a backport PR to the most recent stable branch on merge.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Replace references to master with references to main in maintainers_guide.rst

5 participants