Code Monkey home page Code Monkey logo

Comments (7)

dsame avatar dsame commented on July 4, 2024 2

Hi @IOrlandoni,
Ok, i preferred to follow the common pattern and call the rate limit API directly in this PR
Please take a look at the sample build output to make sure it is that you expected:

debug status before the processing: https://github.com/akv-demo/test-stale/actions/runs/5257429320/jobs/9500195867#step:2:40

debug status after the processing: https://github.com/akv-demo/test-stale/actions/runs/5257429320/jobs/9500195867#step:2:208

and summary of the usage logged on info level:
https://github.com/akv-demo/test-stale/actions/runs/5257429320/jobs/9500195867#step:2:209

from stale.

dsame avatar dsame commented on July 4, 2024

Hello @IOrlandoni , thank you for you suggestion. I looks very useful and will be implemented asap

from stale.

IOrlandoni avatar IOrlandoni commented on July 4, 2024

Hey @dsame, just wanted to confirm if you're working on it. If you are, I wont :)

from stale.

dsame avatar dsame commented on July 4, 2024

Hi @IOrlandoni , i am already close to providing the solution, but i'd like to see you approach because this statement "The information should already be included in the headers of the responses from Github" is not clear for me. According to the docs the response does not include api usage info: see response schema

from stale.

IOrlandoni avatar IOrlandoni commented on July 4, 2024

Hey @dsame. You should not see it in the response schema because it exists on the headers, as per the docs here. I have no idea if you have access to the headers through Octokit though.

Given your knowledge of the action, I doubt I'll be able to provide a better solution than yours...!

from stale.

IOrlandoni avatar IOrlandoni commented on July 4, 2024

@dsame LGTM. Can't complain! Thank you!

from stale.

dsame avatar dsame commented on July 4, 2024

Hello @IOrlandoni , the related PR is merged and will be available in the next release.

Meanwhile you can test and review the change by switching to the main branch

- uses: actions/stale@main

from stale.

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.