Skip to content

Ignore builds on Crowdin PRs#6037

Merged
tramuntanal merged 1 commit intodevelopfrom
chore/ignore-builds-on-translations
Apr 29, 2020
Merged

Ignore builds on Crowdin PRs#6037
tramuntanal merged 1 commit intodevelopfrom
chore/ignore-builds-on-translations

Conversation

@mrcasals
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

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

@mrcasals
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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.

3 participants