Skip to content
This repository has been archived by the owner on May 25, 2023. It is now read-only.

fix: update pr-compliance-action #1303

Merged

Conversation

mtfoley
Copy link
Contributor

@mtfoley mtfoley commented Nov 12, 2021

What type of PR is this? (check all applicable)

  • ♻️ Refactor
  • ✨ Feature
  • 🐛 Bug Fix
  • 👷 Optimization
  • 📝 Documentation Update
  • 🔖 Release
  • 🚩 Other

Description

Update pr-compliance-action to a version that should strip HTML comments out of PR description so it doesn't find false positives.

This PR fixes #1302

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings

Added tests?

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help

Added to documentation?

  • 📜 readme
  • 📜 contributing.md
  • 📓 docs
  • 📕 storybook
  • 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

sigh

@0-vortex
Copy link
Contributor

LGTM!

@0-vortex 0-vortex merged commit ba6c0fe into open-sauced:main Nov 12, 2021
github-actions bot pushed a commit that referenced this pull request Nov 12, 2021
### [0.39.1](v0.39.0...v0.39.1) (2021-11-12)

### Bug Fixes

* update pr-compliance-action ([#1303](#1303)) ([ba6c0fe](ba6c0fe)), closes [#1302](#1302)
@github-actions
Copy link
Contributor

🎉 This PR is included in version 0.39.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

Bug: pr-compliance-action sees issue references inside HTML comments
2 participants