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.

Sign up for GitHub

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

Jump to bottom

Fix copyright dates from PR #728 for 2024 #737

Merged
merged 1 commit into from
Jan 8, 2024
Merged

Fix copyright dates from PR #728 for 2024 #737

merged 1 commit into from
Jan 8, 2024
+4 −4

Conversation

woodsp-ibm
Copy link
Member

Summary

#728 was created last year but upon merge earlier today I guess the last updated dates of the files are now 2024 when the branch of my fork, and the files that were modified at the end of 2023, were merged into main. Indeed just pulling main and running the copyright check now fails for these files. This PR simply updates the copyright dates for these so CI will pass.

Wrong Copyright Year:'test/neural_networks/test_estimator_qnn.py':  Current:'# (C) Copyright IBM 2022, 2023.' Correct:'# (C) Copyright IBM 2022, 2024.'
Wrong Copyright Year:'qiskit_machine_learning/neural_networks/neural_network.py':  Current:'# (C) Copyright IBM 2020, 2023.' Correct:'# (C) Copyright IBM 2020, 2024.'
Wrong Copyright Year:'qiskit_machine_learning/neural_networks/estimator_qnn.py':  Current:'# (C) Copyright IBM 2022, 2023.' Correct:'# (C) Copyright IBM 2022, 2024.'
Wrong Copyright Year:'qiskit_machine_learning/neural_networks/sampler_qnn.py':  Current:'# (C) Copyright IBM 2022, 2023.' Correct:'# (C) Copyright IBM 2022, 2024.'

Details and comments

@woodsp-ibm woodsp-ibm added the stable backport potential The bug might be minimal and/or import enough to be port to stable label Jan 8, 2024
adekusar-drl
adekusar-drl approved these changes Jan 8, 2024
View reviewed changes
Copy link
Collaborator

@adekusar-drl adekusar-drl left a comment

Choose a reason for hiding this comment

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

I hope it will pass the CI checks.

@woodsp-ibm woodsp-ibm mentioned this pull request Jan 8, 2024
Merged
@woodsp-ibm woodsp-ibm mentioned this pull request Jan 8, 2024
Merged
@mergify mergify bot merged commit 1203346 into qiskit-community:main Jan 8, 2024
17 checks passed
mergify bot pushed a commit that referenced this pull request Jan 8, 2024
@woodsp-ibm @mergify
Fix copyright dates from PR #728 for 2024 (#737)
834bb5e 
(cherry picked from commit 1203346)
@mergify mergify bot mentioned this pull request Jan 8, 2024
Merged
@woodsp-ibm woodsp-ibm deleted the fix_dates branch January 8, 2024 20:48
mergify bot added a commit that referenced this pull request Jan 8, 2024
@mergify @woodsp-ibm
Fix copyright dates from PR #728 for 2024 (#737) (#738)
0ef6dfe 
(cherry picked from commit 1203346)

Co-authored-by: Steve Wood <[email protected]>
oscar-wallis pushed a commit that referenced this pull request Feb 16, 2024
@woodsp-ibm
Fix copyright dates from PR #728 for 2024 (#737)
215e878
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adekusar-drl adekusar-drl adekusar-drl approved these changes

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

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

@oscar-wallis oscar-wallis Awaiting requested review from oscar-wallis oscar-wallis is a code owner

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

@Benjamin-Symons Benjamin-Symons Awaiting requested review from Benjamin-Symons Benjamin-Symons is a code owner

Assignees
No one assigned
Labels
automerge stable backport potential The bug might be minimal and/or import enough to be port to stable
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@woodsp-ibm @adekusar-drl