Giter Site home page Giter Site logo

Comments (7)

staabm avatar staabm commented on May 21, 2024 1

Hmm I would prefer a more api centric approach

from phpstan-todo-by.

staabm avatar staabm commented on May 21, 2024 1

issue/pr closed state handling was implemented with #62

as you pointed out the 'released' state is not easily to determine. until github.com provides a proper mean to detect this state I will close this issue for now.

thanks for the feedback though

from phpstan-todo-by.

staabm avatar staabm commented on May 21, 2024

We could implement a todo-by-ticket github-issues fetcher todo that.

This should work for github issues and PRs - these are very similar on github

Just enhance the rule implemented with #26 with a new fetcher

Since github issue identifiers have a different pattern then jira issue keys, we might need a separate rule

from phpstan-todo-by.

nikophil avatar nikophil commented on May 21, 2024

sadly, what I was thinking seems to be not doable easily: the information of the first git tag which includes the PR, which you can view in GitHub interface, is not exposed in the API.

I can't find any easy way to guess this information.
The best which can be done is to expire todo comment if the issue is resolved or the PR is merged, but IMO it's less useful than the logic based on versions

from phpstan-todo-by.

staabm avatar staabm commented on May 21, 2024

the refined-github browser extensions renders a note in PRs which have been merged and are part of a PR:

grafik

https://github.com/refined-github/refined-github/blob/a101ac6513d71173d4c5788b999129d812823046/source/features/closing-remarks.tsx#L78-L91

you might find some inspiration there

from phpstan-todo-by.

nikophil avatar nikophil commented on May 21, 2024

haha I didn't even know this information was added by this addon, I use it since too much time 😅

https://github.com/refined-github/refined-github/blob/a101ac6513d71173d4c5788b999129d812823046/source/features/closing-remarks.tsx#L21-L31

they look into this url and hack into the html
https://github.com/staabm/phpstan-todo-by/branch_commits/98d37d9dcfbcf1a70bb7a36ecea7c6ca7430c347

it seems there is no API equivalent. This solution is a bit hacky: I think there is no BC policy on this type of html url. That why from time to time the add-on has bugs: when GH changes its urls / or html markdown.
But if you're still okay.... why not ;)

from phpstan-todo-by.

github-actions avatar github-actions commented on May 21, 2024

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

from phpstan-todo-by.

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.