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.

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

[COMPLIANCE] Add Copyright and License Headers #17117

Closed
wants to merge 1 commit into from

Conversation

hashicorp-copywrite[bot]
Copy link
Contributor

@hashicorp-copywrite hashicorp-copywrite bot commented May 8, 2023

Hi there 👋

This PR was auto-generated as part of an internal review of public repositories that are not in compliance with HashiCorp's licensing standards.

Frequently Asked Questions

Why am I getting this PR? This pull request was created because one or more source code files were found missing copyright and/or license headers.

More info is available in the RFC

How do you determine which files should get copyright headers? Attempts are made to skip scanning autogenerated files (e.g., `go.mod`) and prose. If you find file types you feel should be excluded from future scans, please reach out to:

#proj-software-copyright

I have a file or folder which should be exempted, how do I do that? You may exempt certain files or folders from being scanned by adding a `.copywrite.hcl` config in the root of your repo. You can use the [`copywrite init`](https://go.hashi.co/copywrite) command to interactively create a config for this project.

An example schema can be found below. Add a doublestar**-capable pattern to the header_ignore list to skip it in future scans.

# (OPTIONAL) Overrides the copywrite config schema version
# Default: 1
schema_version = 1

project {
  # (OPTIONAL) SPDX-compatible license identifier
  # Leave blank if you don't wish to license the project
  # Default: "MPL-2.0"
  # license = ""

  # (OPTIONAL) Represents the year that the project initially began
  # Default: <the year the repo was first created>
  # copyright_year = 0

  # (OPTIONAL) A list of globs that should not have copyright or license headers .
  # Supports doublestar glob patterns for more flexibility in defining which
  # files or folders should be ignored
  # Default: []
  header_ignore = [
    # "vendors/**",
    # "**autogen**",
  ]
}

More information about configuration options is available in [the documentation](https://github.com/hashicorp/copywrite#config-structure).
I added a config. How do I trigger this PR to be rebased? HashiCorp employees can use the [Copywrite SlackBot](https://hashicorp.slack.com/archives/D052WARFFS8) to trigger a rebase. You can DM the slackbot with "headers nomad" to trigger a PR rebasing.
Why don't the headers include a copyright date? Copyright headers are not required to include a year. In the interest of pragmatism, HashiCorp has decided to exclude the year from headers and instead list it in the LICENSE file at the root of the repository instead.

Additional FAQs are available at https://go.hashi.co/header-faq

Please approve and merge this PR in a timely manner to keep this source code compliant with our OSS license agreement. If you have any questions or feedback, reach out to #proj-software-copyright.

Thank you!


Powered by copywrite, made with ❤️ by @hashicorp

@github-actions
Copy link

github-actions bot commented May 8, 2023

Ember Asset Size action

As of d5904cf

Files that got Bigger 🚨:

File raw gzip
nomad-ui.js +133 B +8 B

Files that stayed the same size 🤷‍:

File raw gzip
vendor.js 0 B 0 B
nomad-ui.css 0 B 0 B
vendor.css 0 B 0 B

@github-actions
Copy link

github-actions bot commented May 8, 2023

Ember Test Audit comparison

main cdc7ac1 change
passes 1475 1475 0
failures 0 0 0
flaky 0 0 0
duration 11m 33s 234ms 11m 15s 849ms -17s 385ms

@shoenig
Copy link
Member

shoenig commented May 8, 2023

@philrenaud is this gonna break the same as f74623a ?

@philrenaud
Copy link
Contributor

@shoenig Yeah, this'll break the streaming-file.hbs template at the least. I think @lgfa29 's fix @ hashicorp/copywrite#59 should have taken care of this, but maybe the script hasn't been updated with their latest main. Want me to make a manual change to this PR?

@shoenig
Copy link
Member

shoenig commented May 30, 2023

Oh nice I didn't know @lgfa29 had a fix. I can try just re-running it manually and replacing this robot PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants