Skip to content

Remove response.data.status check#25

Closed
sargunv wants to merge 2 commits intojitterbit:masterfrom
sargunv:patch-1
Closed

Remove response.data.status check#25
sargunv wants to merge 2 commits intojitterbit:masterfrom
sargunv:patch-1

Conversation

@sargunv
Copy link

@sargunv sargunv commented Mar 15, 2021

It's perfectly valid for a head to be "behind" a base when comparing commits for pull request events. For example, PRing a feature branch of main while other commits have landed on main after the feature branch was started but before the PR was created, or a PR to merge a "development" branch onto a slower moving "release" branch that has some hotfixes on it.

sargunv added 2 commits March 15, 2021 14:34
It's perfectly valid for a head to be "behind" a base when comparing commits for pull request events. For example, PRing a feature branch of `main` while other commits have landed on `main` after the feature branch was started but before the PR was created, or a PR to merge a "development" branch onto a slower moving "release" branch that has some hotfixes on it.
Ana06 added a commit to Ana06/get-changed-files that referenced this pull request Jun 4, 2021
I have cherry-picked the removal of the ahead check:
jitterbit#25

This seems to be a change Jitterbit's community agrees on:
jitterbit#17

Build this change and prepare v1.2 release.
@Ana06
Copy link

Ana06 commented Jun 4, 2021

Nice change @sargunv! I have cherry-picked 54f3703 and released it in Ana06/get-changed-files@v1.2 because we want to use it in capa. But this seems to be a change that the community want: #11, #19, so I hope this makes it upstream. In case it doesn't (or until it does), feel free to use Ana06/get-changed-files@v1.2 (which also supports pull_request_target and allow to filter files using regular expressions).

@dhimanmohit2548
Copy link

Any progress on this one?

@sargunv
Copy link
Author

sargunv commented Sep 9, 2021

Seems like this isn't merging, closing to clear my queue

@sargunv sargunv closed this Sep 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants