Giter VIP home page Giter VIP logo

Comments (7)

LironEr avatar LironEr commented on June 2, 2024

See here: #115 (comment)

Forgot to document this ability, but let me know if this solves the issue for you

from bundlemon.

zerofirework avatar zerofirework commented on June 2, 2024

Yes. It's amazing!

Quick question about the behavior, if I set the limit by max increase percent and I approve the change that increases the binary size, will the next PR fail the bundlemon CI again since the baseline didn't change and the binary size will stay high?

from bundlemon.

LironEr avatar LironEr commented on June 2, 2024

yes, each approval is for a specific commit, the baseline will always be the target branch, once you merge the baseline will change.

from bundlemon.

zerofirework avatar zerofirework commented on June 2, 2024

I found two issues when trying this feature:

  1. After I approve the change, the bundlemon CI job still fails. I was expecting it to pass.
  2. There is a race condition in the UI between when the review button checks auth state and the auth state changes. If auth state changes after the review button checks the auth status, it will show me log in is required even if I have logged in.

from bundlemon.

LironEr avatar LironEr commented on June 2, 2024

The approval applies to a specific commit that goes to GitHub and changes the current commit status from failed to success, re-running or a new commit will cause it to fail again.

We can think of a solution that checks the PR labels and auto-approves new jobs.

For now, you can do that bundlemon wont fail the pipeline

- name: Run BundleMon
   uses: lironer/bundlemon-action@v1
   with:
     fail-on-error: false

Then your job will always be successful, and if you want to verify the bundlemon status have a "required" status of the bundlemon commit status

from bundlemon.

zerofirework avatar zerofirework commented on June 2, 2024

Yeah, having a way to auto approve the jobs with a PR labels can be a great idea.

Do you mind elaborating what you meant by "have a "required" status of the bundlemon commit status"

Thanks for the reply!

from bundlemon.

LironEr avatar LironEr commented on June 2, 2024
  1. Setup bundlemon github-integration, then on every commit in a PR bundlemon will add a commit status

image

  1. In branch protection rules, make BundleMon a required status check
    image

When one of the defined limits exceeds BundleMon will create a failed commit status, You can decide if you want to approve it via the BundleMon website. If you approve it will change the commit status to success

from bundlemon.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.