Skip to content
This repository has been archived by the owner on Aug 19, 2023. It is now read-only.

Note banner in README file #1790

Merged
merged 2 commits into from
Aug 18, 2023
Merged

Note banner in README file #1790

merged 2 commits into from
Aug 18, 2023

Conversation

1ucian0
Copy link
Member

@1ucian0 1ucian0 commented Aug 4, 2023 •
edited by jakelishman
Loading

Add a banner with some context for the archiving.

Screenshot 2023-08-04 at 23 20 25

Fix #1793

jakelishman
jakelishman reviewed Aug 7, 2023
View reviewed changes
Copy link
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.

This largely looks fine to me, but let's leave it unmerged until it can be the last commit before the repo's archiving. Did you mean to remove the header banner image?

@1ucian0
Copy link
Member Author

1ucian0 commented Aug 18, 2023

Did you mean to remove the header banner image?

Yes. to bring the banner up and "reduce the importance" of the repo.

@1ucian0 1ucian0 merged commit eb5ce8a into Qiskit:master Aug 18, 2023
8 checks passed
@1ucian0 1ucian0 mentioned this pull request Aug 18, 2023
Closed
21 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jakelishman jakelishman jakelishman approved these changes

@mtreinish mtreinish Awaiting requested review from mtreinish mtreinish is a code owner

@kevinhartman kevinhartman Awaiting requested review from kevinhartman kevinhartman is a code owner

@javabster javabster Awaiting requested review from javabster javabster is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update README with notice of archiving
2 participants
@1ucian0 @jakelishman