Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Cypress CI process while opening PR from a fork #29098

Merged
merged 2 commits into from
Jun 6, 2024

Conversation

mistercrunch
Copy link
Member

@mistercrunch mistercrunch commented Jun 5, 2024

As observed in #28774 (thanks for catching this @rusackas), I broke the PR-from-fork in my recently merged PR here #29077 .

This PR removes special handling of ref: on github_event in (pull_request, push)

@github-actions github-actions bot added the github_actions Pull requests that update GitHub Actions code label Jun 5, 2024
@mistercrunch mistercrunch changed the title chore: test Cypress process while opening PR from a fork fix: Cypress CI process while opening PR from a fork Jun 5, 2024
@mistercrunch mistercrunch marked this pull request as ready for review June 5, 2024 23:09
@@ -14,8 +14,6 @@
# KIND, either express or implied. See the License for the
# specific language governing permissions and limitations
# under the License.
"""Package's main module!"""
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I removed this silly string to trigger CI as it only runs certain steps based on which folders/files have changed

Copy link

codecov bot commented Jun 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.31%. Comparing base (76d897e) to head (0bcd0b7).
Report is 283 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #29098      +/-   ##
==========================================
+ Coverage   60.48%   70.31%   +9.82%     
==========================================
  Files        1931     1951      +20     
  Lines       76236    77611    +1375     
  Branches     8568     8756     +188     
==========================================
+ Hits        46114    54571    +8457     
+ Misses      28017    20915    -7102     
- Partials     2105     2125      +20     
Flag Coverage Δ
hive 48.94% <ø> (-0.23%) ⬇️
javascript 57.83% <ø> (+0.12%) ⬆️
mysql 77.19% <ø> (?)
postgres 77.33% <ø> (?)
presto 53.53% <ø> (-0.27%) ⬇️
python 83.67% <ø> (+20.18%) ⬆️
sqlite 76.77% <ø> (?)
unit 59.03% <ø> (+1.41%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mistercrunch mistercrunch merged commit 8f02eb0 into apache:master Jun 6, 2024
44 of 65 checks passed
@mistercrunch mistercrunch deleted the test_fork branch June 6, 2024 00:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
github_actions Pull requests that update GitHub Actions code size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants