Skip to content

Revert "Don't schedule CI jobs for locales PRs (#7534)"#7546

Merged
mrcasals merged 1 commit intodevelopfrom
revert/7534
Mar 5, 2021
Merged

Revert "Don't schedule CI jobs for locales PRs (#7534)"#7546
mrcasals merged 1 commit intodevelopfrom
revert/7534

Conversation

@mrcasals
Copy link
Copy Markdown
Contributor

@mrcasals mrcasals commented Mar 5, 2021

🎩 What? Why?

This PR reverts #7534 as it's breaking the pipelines for PRs coming from forks.

📌 Related Issues

None

Testing

None.

@mrcasals mrcasals merged commit 0d69358 into develop Mar 5, 2021
@mrcasals mrcasals deleted the revert/7534 branch March 5, 2021 07:32
@oliverbarnes
Copy link
Copy Markdown
Contributor

How is it breaking? Would be good to have this documented here

@mrcasals
Copy link
Copy Markdown
Contributor Author

mrcasals commented Mar 5, 2021

For some reason it's not starting the CI jobs for PRs coming from forks. I guess it's due to the removal of the pull_request trigger.

At some point we could try reapplying the path, but also keeping the pull_request part:

on:
  push:
    branches:
      - develop
      - release/*
      - "*-stable"
  pull_request:

env:
  ...

@oliverbarnes
Copy link
Copy Markdown
Contributor

Ok, I see. Yeah, that makes sense. Completely missed that in my review, I only saw the branch changes

entantoencuanto added a commit that referenced this pull request Mar 5, 2021
* develop:
  Fix infinite loop when impersonated session time runs out (#7221)
  New Crowdin updates (#7543)
  Migrate Admin menus to Menu Registry Part 2 (#7382)
  Replace xls with xlsx (#7421)
  Use cache_key_with_version instead of cache version (#7532)
  Add support for ElectionGuard voting scheme (#7454)
  Fix record encryptor trying to decrypt empty strings (#7542)
  Revert "Don't schedule CI jobs for locales PRs (#7534)" (#7546)
  New Crowdin updates (#7540)
  New Crowdin updates (#7539)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants