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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ignore builds on Crowdin PRs #6037

Merged
merged 1 commit into from Apr 29, 2020

Conversation

mrcasals
Copy link
Contributor

馃帺 What? Why?

We're seeing a lot of GitHub Actions builds from Crowdin PRs, and they're slowing down the development. Right now, we have 52 pages of enqueued actions (52*25 = 1300 actions), out of which 29 pages are from the l10n_develop branch.

image

This PR ignores the builds from any PR whose branch matches the l10n_* pattern.

馃搶 Related Issues

None

馃搵 Subtasks

None

@mrcasals mrcasals self-assigned this Apr 28, 2020
@mrcasals
Copy link
Contributor Author

@decidim/core can you check this, please? What do you think about it?

@mrcasals
Copy link
Contributor Author

Note that this won't have any effect until this is merged, ported to any relevant branch (0.21-stable for example) and the current Crowdin PRs are merged!

Copy link
Contributor

@microstudi microstudi left a comment

Choose a reason for hiding this comment

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

thanks

@tramuntanal tramuntanal merged commit 8d89b60 into develop Apr 29, 2020
@tramuntanal tramuntanal deleted the chore/ignore-builds-on-translations branch April 29, 2020 06:31
@mrcasals
Copy link
Contributor Author

Since at this moment we're still running tests from Crowdin PRs on l10n_0.21-stable, I think it's worth porting this PR to 0.21-stable and merge all locales PRs we currently have.

What do you think? @decidim/core

@tramuntanal
Copy link
Contributor

You are totally right, we're doing it

ace pushed a commit to aspgems/decidim that referenced this pull request May 5, 2020
* develop: (29 commits)
  Update Conversations design with decidim-design UI (decidim#6008)
  Add counter of active users to admin dashboard (decidim#5907)
  Show activity graphs on admin dashboard (decidim#6030)
  Update sassc gem version (decidim#6062)
  Fix generator Gemfile after puma upgrade (decidim#6060)
  New Crowdin translations (decidim#6059)
  Add Slovak as a new language (decidim#6039)
  Remove all tests for i18n PRs (decidim#6061)
  Update move up and down buttons after dragging questions when managing questionnaire (decidim#5947)
  Fix using Decidim as a provider for omniauth authentication (decidim#6042)
  Add redesign for responsive public profile navigation tabs (decidim#6032)
  Add versioning pages to initiatives (decidim#5935)
  Notify users when roles get assigned (decidim#5886)
  Improve the budget page and the project card (decidim#5809)
  New Crowdin translations (decidim#6050)
  New Crowdin translations (decidim#6046)
  Ignore jobs on locales branches (decidim#6047)
  Automatic task for deleting Meeting Inscription data (decidim#5989)
  New Crowdin translations (decidim#5877)
  Ignore builds on Crowdin PRs (decidim#6037)
  ...
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.

None yet

3 participants