Skip to content

Commit

Permalink
Merge pull request #29 from juanigalan91/feat/update-comment
Browse files Browse the repository at this point in the history
feat(comments): update existing comment instead of creating a new one
  • Loading branch information
juanigalan91 authored Jul 10, 2021
2 parents a1e73cb + adabd49 commit c39cf4f
Show file tree
Hide file tree
Showing 8 changed files with 2,271 additions and 2,165 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/monorepolyser.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ jobs:
uses: ./
with:
impact-analysis: true
high-impact-threshold: 50
high-impact-threshold: 10
on-high-impact: 'comment,add-labels'
high-impact-labels: 'high-impact'
verbose: 'comment'
Expand Down
10 changes: 5 additions & 5 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,15 +11,15 @@ jobs:
- name: "Checkout repository"
uses: actions/checkout@v2
- name: "Setup node env"
uses: actions/setup-node@c6fd00c #v2.1.2
uses: actions/setup-node@c6fd00ceb9747fb23ffdf72987450a2664414867 #v2.1.2
with:
node-version: '13.7.0'
- name: "Get cache hash"
id: cache-prep
run: echo "::set-output name=hash::"$(cat yarn.lock {packages}/**/package.json | sha1sum | egrep -o '^\w+')
- name: "Cache dependencies"
id: cache
uses: actions/cache@70655ec #v1.1.2
uses: actions/cache@70655ec8323daeeaa7ef06d7c56e1b9191396cbe #v1.1.2
with:
path: node_modules
key: ${{ runner.os }}-dependency-hash-${{ steps.cache-prep.outputs.hash }}
Expand All @@ -31,16 +31,16 @@ jobs:
needs: install-dependencies
steps:
- name: "Checkout repository"
uses: actions/checkout@722adc6 #v2
uses: actions/checkout@v2 #v2
- name: "Setup node env"
uses: actions/setup-node@c6fd00c #v2.1.2
uses: actions/setup-node@c6fd00ceb9747fb23ffdf72987450a2664414867 #v2.1.2
with:
node-version: '13.7.0'
- name: "Get cache hash"
id: cache-prep
run: echo "::set-output name=hash::"$(cat yarn.lock {packages}/**/package.json | sha1sum | egrep -o '^\w+')
- name: "Cache dependencies"
uses: actions/cache@70655ec #v1.1.2
uses: actions/cache@70655ec8323daeeaa7ef06d7c56e1b9191396cbe #v1.1.2
with:
path: node_modules
key: ${{ runner.os }}-dependency-hash-${{ steps.cache-prep.outputs.hash }}
Expand Down
6 changes: 6 additions & 0 deletions CHANGE_LOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,12 @@

## Unreleased

## 0.2.7
- Update existing PR comment instead of creating a new one in order to avoid spamming the PR

## 0.2.6
- Allows to pass in a regular expression to manually add packages to the high impact analysis

## 0.2.5
- Fix bug when the file changed is outside the workspace

Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "monorepolyser",
"version": "0.2.6",
"version": "0.2.7",
"description": "",
"main": "index.js",
"private": true,
Expand Down
4 changes: 2 additions & 2 deletions packages/ga-impact-analysis/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ const main = async (options: ImpactAnalysisOptions) => {
const comment = new Comment();

comment.addTitle({
title: 'Impact Analysis',
title: 'Impact Analysis - Changes',
level: 2,
});
comment.addText({
Expand Down Expand Up @@ -135,7 +135,7 @@ const main = async (options: ImpactAnalysisOptions) => {
verboseComment = new Comment();

verboseComment.addTitle({
title: 'Impact Analysis',
title: 'Impact Analysis - Report',
level: 2,
});

Expand Down
Loading

0 comments on commit c39cf4f

Please sign in to comment.