Skip to content

Add and enforce copyright headers for C# files under /build, /src, and /tools#1445

Merged
zacharycmontoya merged 2 commits into
masterfrom
zach/refactor/apply-copyright-header
May 24, 2021
Merged

Add and enforce copyright headers for C# files under /build, /src, and /tools#1445
zacharycmontoya merged 2 commits into
masterfrom
zach/refactor/apply-copyright-header

Conversation

@zacharycmontoya

@zacharycmontoya zacharycmontoya commented May 4, 2021

Copy link
Copy Markdown
Contributor

Changes proposed in this pull request:

  • Modifies stylecop rules to enforce copyright headers on C# files
  • Runs the automated rule to add the missing copyright headers
  • Affects all subdirectories EXCEPT /test, which currently doesn't have stylecop applied

@DataDog/apm-dotnet

@zacharycmontoya zacharycmontoya self-assigned this May 4, 2021
@zacharycmontoya
zacharycmontoya requested a review from a team as a code owner May 4, 2021 17:52
lucaspimentel
lucaspimentel reviewed May 4, 2021
View reviewed changes
Comment thread test/benchmarks/Benchmarks.Trace/Properties/launchSettings.json
@zacharycmontoya
zacharycmontoya force-pushed the zach/refactor/apply-copyright-header branch from dea75ce to 42b386c Compare May 4, 2021 18:43
@zacharycmontoya
zacharycmontoya force-pushed the zach/refactor/apply-copyright-header branch 2 times, most recently from 3eba4b7 to 42a47e3 Compare May 4, 2021 20:15
@zacharycmontoya
zacharycmontoya force-pushed the zach/refactor/apply-copyright-header branch from 42a47e3 to a0d9f21 Compare May 13, 2021 19:57
@lucaspimentel
lucaspimentel requested a review from a team May 18, 2021 18:48
@zacharycmontoya
zacharycmontoya force-pushed the zach/refactor/apply-copyright-header branch from a0d9f21 to cc7e457 Compare May 24, 2021 22:08
@zacharycmontoya
zacharycmontoya merged commit b68cdfe into master May 24, 2021
@zacharycmontoya
zacharycmontoya deleted the zach/refactor/apply-copyright-header branch May 24, 2021 23:46
@pellared pellared mentioned this pull request May 31, 2021
Merged
@andrewlock andrewlock added this to the 1.27.0 milestone Jun 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lucaspimentel lucaspimentel lucaspimentel approved these changes

Assignees

@zacharycmontoya zacharycmontoya

Labels

type:refactor

Projects

None yet

Milestone

1.27.0

Development

Successfully merging this pull request may close these issues.

3 participants

@zacharycmontoya @lucaspimentel @andrewlock