Skip to content

Fix code style on PRs#6534

Merged
danharrin merged 2 commits into
2.xfrom
fix-code-style-on-prs
May 17, 2023
Merged

Fix code style on PRs#6534
danharrin merged 2 commits into
2.xfrom
fix-code-style-on-prs

Conversation

@zepfietje
Copy link
Copy Markdown
Member

@zepfietje zepfietje commented May 17, 2023

Makes it easier to review PRs when we fix code style on PR already instead of after merge.

  • Changes have been thoroughly tested to not break existing functionality.
  • New functionality has been documented or existing documentation has been updated to reflect changes.
  • Visual changes are explained in the PR description using a screenshot/recording of before and after.

@zepfietje zepfietje changed the title Fix code style on prs Fix code style on PRs May 17, 2023
@zepfietje zepfietje added the task label May 17, 2023
@zepfietje zepfietje added this to the v2 milestone May 17, 2023
@what-the-diff
Copy link
Copy Markdown
Contributor

what-the-diff Bot commented May 17, 2023

PR Summary

  • Update fix-code-style.yml triggers
    The fix-code-style.yml file now runs on both push and pull_request events.
  • Modify npm-build.yml to run on PR closed
    The npm-build.yml file now runs when a pull request is closed, not just when it's merged.

@danharrin danharrin merged commit bda92c5 into 2.x May 17, 2023
@danharrin danharrin deleted the fix-code-style-on-prs branch May 17, 2023 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants