1
0
mirror of synced 2026-01-01 09:04:46 -05:00
Commit Graph

734 Commits

Author SHA1 Message Date
Ramya Parimi
03c9f5ea57 Merged notification workflow are failing (#24081)
* Update triage-issue-comments.yml

* Update merged-notification.yml

* Update triage-issue-comments.yml
2022-01-07 02:30:47 +00:00
James M. Greene
2d8a3cc7d0 Add actions:read permissions to GITHUB_TOKEN (#24113) 2022-01-06 17:16:19 +00:00
James M. Greene
be4249b57c Remove the concept of "undeployment" as a part of the PR deployment lifecycle (#23731)
* Update remove-stale-staging-resources workflow to completely replace undeploy workflow
* Delete the staging-undeploy-pr workflow file
* Delete all undeployment scripts and logic
* Remove all references to the automated-block-deploy label used for undeployment
* Simplify staging cross-workflow concurrency needs
2022-01-06 16:27:48 +00:00
Peter Bengtsson
26d2c5438a Check for orphaned assets in CI (#24074) 2022-01-05 16:26:45 +00:00
Mike Surowiec
a840d2bb13 Add production Azure workflow (#24038)
* add prod-build-deploy-azure workflow
2022-01-04 22:40:08 -06:00
Peter Bengtsson
9bdbec73d4 escape echo of changed files in Action (#24058)
* escape echo of changed files in Action

* escaped

* escaped (2)

* temporary

* reset temporary tests
2022-01-04 22:50:43 +00:00
Robert Sese
a9da0c8681 Singular script directory (#24052) 2022-01-04 21:00:47 +00:00
Sarah Edwards
7cff1636cd use pull_request_target trigger (#23946)
Co-authored-by: Kevin Heis <heiskr@users.noreply.github.com>
2022-01-04 19:56:16 +00:00
James M. Greene
6f2170e825 Remove duplicate CI env vars from Actions workflows (#23921) 2022-01-04 18:29:04 +00:00
Peter Bengtsson
1a8d06089e use pull_request for the private staging-build-and-deploy (#23783) 2022-01-04 18:01:27 +00:00
Kevin Heis
55ee70e06b Check that actions specify hashes instead of allowlist (#24042)
* Check that actions specify hashes instead of allowlist

* Fixes for unhashed version

* Update actions-workflows.js
2022-01-04 17:43:40 +00:00
Sarah Edwards
767d7d8f51 Get all open PRs instead of last 100 (#23985) 2022-01-04 16:16:58 +00:00
Marcelo Jacobus
3b0594e3ce Automate closing of stale translation batches (#23581)
* Automatically close stale translation batches

* fix: adjust spacing for the query key

Resolves CI failure: https://github.com/github/docs-internal/runs/4486848450?check_suite_focus=true#step:3:4

* Fix query

* enhancement: add type filter to query

`type` filter restricts results to either issues or pull requests. In this case, we want to close existing stale batches that are _only_ pull requests.

Docs: https://docs.github.com/en/search-github/searching-on-github/searching-issues-and-pull-requests#search-only-issues-or-pull-requests

* feat: use lee-dohm/close-matching-issues v2.1.0

https://github.com/lee-dohm/close-matching-issues/pull/357 is merged into this release, which gives the action the ability to close matching issues _and_ pull requests.

Co-authored-by: Francis <15894826+francisfuzz@users.noreply.github.com>
2021-12-20 17:50:37 +00:00
James M. Greene
94659705b1 Alternative repo-sync fixes (#23892)
* Info is in the data prop

* Don't bother to comment when closing our own automated repo-sync PRs

Co-authored-by: Robert Sese <sese@github.com>
2021-12-18 14:31:40 -06:00
James M. Greene
b486966547 Add a few checks to close the repo-sync PR if no changes remain (#23839) 2021-12-17 20:37:56 +00:00
James M. Greene
d6405ecb45 Optimize workflow runs with concurrency settings (#23811)
* Add concurrency settings for a bunch of workflows
* Restrict prod deployments to one at a time but disallow interrupting
* Remove redundant pull_request.types specification
2021-12-17 16:00:52 +00:00
Peter Bengtsson
a44fe743f7 don't run old link checkers on pushes (#23785) 2021-12-16 18:10:20 +00:00
Rachael Sewell
02c5e44c4d Run translation workflow once a day (#23765)
Co-authored-by: Francis <15894826+francisfuzz@users.noreply.github.com>
2021-12-16 16:56:35 +00:00
Peter Bengtsson
84f323fdc1 Ability to soft-purge Fastly with a click of a button (#23508)
* Ability to soft-purge Fastly with a click of a button

* permissions

* install dependencies would help
2021-12-16 13:30:22 +00:00
Peter Bengtsson
17e33bbb61 don't run unit tests on pushes to main (#23755) 2021-12-16 12:51:30 +00:00
Peter Bengtsson
1c2219c057 remove old debugging in staging-deploy-pr (#23701) 2021-12-15 20:16:04 +00:00
Peter Bengtsson
bf5c7353d2 eslint check the .github/actions-scripts/* files too (#23698)
* eslint check the .github/actions-scripts/* files too

* but ignore node_modules/.bin/*

* fix errors

* last one

* feedbacked
2021-12-15 17:35:34 +00:00
Peter Bengtsson
a736b53cec Only run browser tests on pull request (#23704)
* only run browser tests on pull_request

* package.json also

* helps if I edit the right file
2021-12-15 17:21:51 +00:00
Robert Sese
0e9c08f040 Unlock before approving in repo sync (#23730)
* Unlock before approving

* Revert "Unlock before approving"

This reverts commit 2b9bc2de94b412a49d3b81f4e48d798c48abfca0.

* Set content:write so we can approve locked PRs
2021-12-15 17:01:33 +00:00
James M. Greene
dc756b9dd3 Adjust internal staging build/deploy job name (#23725)
A unique name is critical so we can make it a required check
2021-12-14 18:21:47 -06:00
Peter Bengtsson
2e9128b148 shared .next/cache in staging-build workflow (#23700)
Co-authored-by: James M. Greene <JamesMGreene@github.com>
2021-12-15 00:01:12 +00:00
James M. Greene
526b5bd651 Add nextjs caching into all workflows with builds (#23654)
* Add nextjs caching into all workflows with builds

* Reposition the nextjs cache step for test-windows.yml

* Fix cache and run build for browser-test.yml

* Update .github/workflows/browser-test.yml
2021-12-14 23:21:06 +00:00
Peter Bengtsson
f68eb74d90 Make a dedicated, fast, workflow just for docs-internal only (#23289)
* Make a dedicated, fast, workflow just for docs-internal only

Part of #1297

* make staging-build-pr only for github/docs

* prune later

* make it louder and clearer about disabling workflows

* does it merge?

* typo

* rename ref

* rename

* early access should be good to go

* far from perfect

* start with that

* gzip

* rearrange

* html_url

* correction of actions/checkout sha

* correction of actions/setup-node sha

* quote

* ooops

* actually deploy

* move @octokit/rest to dependencies

* await-sleep hack

* reinstall npm

* typo

* CONTEXT_NAME

* deployments:write permission

* pull-requests:read permission

* actions:read and statuses:write permissions

* private repo mention exception

* it's called github.run_id

* Apply suggestions from code review

Co-authored-by: James M. Greene <JamesMGreene@github.com>

* make CONTEXT_NAME optional (if it works)

* comment out CONTEXT_NAME

* simplifying

* going to run on on.pull_request instead

* remove comment

* only the 2-phase staging deploy on github/docs

* better if statement on label check

* refactor of staging-deploy script

* switch to npm install to get the deDependencies back

* using --only=dev

* updating comments

* event_name

* not on pushes to main

* add staging-commit-status-success

* testing testing

* fix linting error

* Remove other docs-internal references from staging-deploy-pr.yml

* Cleaning up new staging-commit-status-success script and usage

* Remove unnecessary environment refs

* Remove unnecessary fallback

Since the only event trigger is pull_request now instead of also push

* Remove unnecessary env vars from workflow

* docs-internal or docs but not both

* Don't provide unnecessary environment refs

* remove now moot exception

* setting it to pull_request_target

Co-authored-by: James M. Greene <JamesMGreene@github.com>
2021-12-14 08:57:54 -05:00
James M. Greene
6b01293461 Add missing security-events:write permissions for CodeQL (#23676) 2021-12-13 23:35:03 -06:00
James M. Greene
3d88cae50b Add permission matrices to all Actions workflows (#23563)
* Add permission matrices to all Actions workflows

Also cleanup a few token references

* Add actions:read permissions for CodeQL

* Add prs:read permissions for unit test workflow
2021-12-14 04:37:36 +00:00
Marcelo Jacobus
dc0d22fa84 Run workflow every six hours (#23585)
* Run cronjob every six hours

* fix: run translation batch PRs on the 25th minute

Resolves CI failure in https://github.com/github/docs-internal/runs/4490412418?check_suite_focus=true#step:10:97

Co-authored-by: Francis <15894826+francisfuzz@users.noreply.github.com>
2021-12-13 17:48:23 +00:00
Marcelo Jacobus
26d7681b95 Remove crowdin.yml workflow (#23490)
* Remove crowdin.yml workflow

Closes https://github.com/github/docs-engineering/issues/1341

* Fix test

Co-authored-by: Francis <15894826+francisfuzz@users.noreply.github.com>
2021-12-13 17:30:07 +00:00
Francis
04c8ed5726 fix: verify Crowdin CLI download before installing (#23515)
This change uses the suggested steps in https://support.crowdin.com/cli-tool/#debian,
with a slight modification in creating the `crowdin.list` file in the `/etc/apt/sources.list.d` directory.

This approach creates a new file for `crowdin.list`, then pipes the content to `tee -a`. Credits go to [this Ask Ubuntu thread reply](https://askubuntu.com/questions/185268/permission-denied-etc-apt-sources-list/185270#185270).
2021-12-11 02:08:00 +00:00
Marcelo Jacobus
daa16775d8 Fix label for translation batches (#23577) 2021-12-11 01:48:22 +00:00
Marcelo Jacobus
953e87e7a0 Use @octomerger for approving and merging translation batches (#23546)
* Use @octomerger for approving and merging translation batches

Here is a [list of bots](https://github.com/github/docs-engineering#bot-accounts)

* Set @octomerger as a code owner of /translations
2021-12-09 20:13:43 +00:00
Marcelo Jacobus
0c02f91dd0 Enable auto-merge for translation-batches (#23464)
* Enable auto-merge for translation-batches

* Attempt to approve and set auto-merge

* Temporarily disable check for tests

* Test commit. Revert later.

* Not self approving anymore

* Improve failure message

* Fix auto-merge

* Remove test code

* Add ownership to @github-actions
2021-12-09 17:06:45 +00:00
Peter Bengtsson
d5a232b0fd delete Docker Action workflow (#23529) 2021-12-09 15:41:43 +00:00
Peter Bengtsson
0035e14e3b one fast link checker to rule them all (#23041)
* one fast link checker to rule them all

* -L alias

* refactor

* hacking away

* passing pages

* adding a workflow

* limit level to critical

* ability to pass a specific set of files

* testing specific files

* remove temporary debugging

* reorganize the printing

* formatting of output

* check images too

* prettyFullPath()

* minimal permission on workflow

* update comment

* new permission

* fix actions shas
2021-12-09 08:56:07 +00:00
docubot
858a0ab79a New translation batch for pt (#23500)
* Delete obsolete files from Crowdin in workflow

* Remove empty files before `crowdin upload`

Those files would have been rejected by Crowdin.
Close https://github.com/github/crowdin-support/issues/117

* Change delete command

* Add crowdin translations

* Run script/i18n/homogenize-frontmatter.js

* Run script/i18n/fix-translation-errors.js

* Run script/i18n/lint-translation-files.js --check parsing

* Run script/i18n/lint-translation-files.js --check rendering

* run script/i18n/reset-files-with-broken-liquid-tags.js --language=pt

* run script/i18n/reset-known-broken-translation-files.js

* Check in pt CSV report

Co-authored-by: Marcelo Jacobus <marcelo.jacobus@gmail.com>
2021-12-08 21:30:15 +00:00
Peter Bengtsson
7b5711e095 cache archived enterproxy proxy responses much longer (#23456)
* cache archived enterproxy proxy responses much longer

* Update middleware/archived-enterprise-versions.js

Co-authored-by: Rachael Sewell <rachmari@github.com>

Co-authored-by: Rachael Sewell <rachmari@github.com>
2021-12-08 19:55:12 +00:00
Peter Bengtsson
ca39429648 Browser test in enja only and cache redirects cache en jajson file (#23502)
* cache the .next/cache directory for faster builds

* browser test in en,ja only and cache .redirects-cache_en_ja.json file

* correction
2021-12-08 19:47:04 +00:00
Peter Bengtsson
b4383a39b1 cache the .next/cache directory for faster builds (#23498) 2021-12-08 14:28:00 -05:00
Marcelo Jacobus
39d07cb977 Restrict workflow to docs-internal (#23489)
Closes https://github.com/github/docs-engineering/issues/1340
2021-12-08 14:13:46 -03:00
Peter Bengtsson
7f6f20a73f run prettier on all code in CI (#23482)
* run prettier on all code in CI

* rename

* test .tsx too

* ignore more

* prettier the stragglers

* minimal permission
2021-12-08 16:11:52 +00:00
Peter Bengtsson
0a7be6644e simulate sync-search in PRs (#22940)
* simulate sync-search in PRs

* Update .github/workflows/sync-search-pr.yml

Co-authored-by: Kevin Heis <heiskr@users.noreply.github.com>

* more consistent naming

Co-authored-by: Octomerger Bot <63058869+Octomerger@users.noreply.github.com>
Co-authored-by: Kevin Heis <heiskr@users.noreply.github.com>
2021-12-07 14:19:07 +00:00
Francis
13a304c14d feat: include label in PR creation (#23380)
This includes the `translation-batch` label when a new PR is created for a translation batch: https://github.com/github/docs-internal/issues?q=label%3Atranslation-batch+
2021-12-06 19:52:02 +00:00
Peter Bengtsson
7d17ce2262 cache .next/cache between unit tests (#23415)
Part of #1246
2021-12-06 18:46:06 +00:00
Robert Sese
a2a0569c84 Prettify (#23374) 2021-12-03 23:44:42 +00:00
Robert Sese
70d7c01dad Skip for repo sync PRs (#23341) 2021-12-02 16:02:57 -06:00
Peter Bengtsson
26014646dd Massive DIFF_FILES crashes bash in repo-sync (#23326)
* Massive DIFF_FILES crashes bash in repo-sync

Part of #1304

* gst

* make exception
2021-12-02 18:57:38 +00:00