Skip to content

Conversation

hebasto
Copy link
Member

@hebasto hebasto commented Aug 30, 2021

This PR:

  • updates the copyright_header.py script to make it handle QML files
  • adds copyright headers to QML files

-BEGIN VERIFY SCRIPT-
git ls-files ./src/qml/pages | xargs -I % ./contrib/devtools/copyright_header.py insert %
-END VERIFY SCRIPT-
Copy link
Contributor

@promag promag left a comment

Choose a reason for hiding this comment

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

Concept ACK

Copy link
Member

@jarolrod jarolrod left a comment

Choose a reason for hiding this comment

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

ACK f82209d

  • f7d3e75

    • crACK: Going by bitcoin/bitcoin@071f2fc and extrapolating the relevant lines, the implementation here is sufficient for adding copyright headers to qml files.
  • f82209d

    • tACK: I made a new branch and dropped this commit. I then ran the script provided and got a zero-diff with this commit.

@hebasto hebasto merged commit b9b1704 into bitcoin-core:main Aug 31, 2021
@hebasto hebasto deleted the 210830-copyright branch August 31, 2021 05:11
hebasto added a commit to hebasto/gui-qml that referenced this pull request Jun 6, 2025
hebasto added a commit to hebasto/gui-qml that referenced this pull request Jun 7, 2025
hebasto added a commit to hebasto/gui-qml that referenced this pull request Jun 7, 2025
hebasto added a commit to hebasto/gui-qml that referenced this pull request Jun 8, 2025
hebasto added a commit to hebasto/gui-qml that referenced this pull request Jun 9, 2025
johnny9 pushed a commit to johnny9/bitcoin-core-app that referenced this pull request Jul 4, 2025
a8c6bf2 scripted-diff: Add copyright headers to QML files (Hennadii Stepanov)
f7d3e75b1f50581af580cc3000e7dfb442b1d96e script: Make copyright_header.py script handle QML files (Hennadii Stepanov)

Pull request description:

  This PR:
  - updates the `copyright_header.py` script to make it handle QML files
  - adds copyright headers to QML files

ACKs for top commit:
  jarolrod:
    ACK a8c6bf2

Tree-SHA512: 051acf344d3a7dea2957266d69281b8eb5cc90d07f6c106ee7a391d22463e530539c01f4b66aca4435077002e214fd620205313390b595ebe95e3ca9016b5c24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants