Commit Graph

161 Commits

Author SHA1 Message Date
dependabot[bot] 5165261a54 chore(deps): bump google-github-actions/deploy-appengine (#6086)
Bumps [google-github-actions/deploy-appengine](https://github.com/google-github-actions/deploy-appengine) from 0.2.0 to 0.8.0.
- [Release notes](https://github.com/google-github-actions/deploy-appengine/releases)
- [Changelog](https://github.com/google-github-actions/deploy-appengine/blob/main/CHANGELOG.md)
- [Commits](https://github.com/google-github-actions/deploy-appengine/compare/v0.2.0...v0.8.0)

---
updated-dependencies:
- dependency-name: google-github-actions/deploy-appengine
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2022-04-20 14:36:14 -07:00
dependabot[bot] f30a64c1bf chore(deps): bump peter-evans/create-pull-request from 3.10.0 to 4.0.2 (#6087)
Bumps [peter-evans/create-pull-request](https://github.com/peter-evans/create-pull-request) from 3.10.0 to 4.0.2.
- [Release notes](https://github.com/peter-evans/create-pull-request/releases)
- [Commits](https://github.com/peter-evans/create-pull-request/compare/9825ae65b1cb54b543b938503728b432a0176d29...bd72e1b7922d417764d27d30768117ad7da78a0e)

---
updated-dependencies:
- dependency-name: peter-evans/create-pull-request
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2022-04-19 15:30:27 -07:00
dependabot[bot] b0612da6f3 chore(deps): bump actions/github-script from 5 to 6 (#6085)
Bumps [actions/github-script](https://github.com/actions/github-script) from 5 to 6.
- [Release notes](https://github.com/actions/github-script/releases)
- [Commits](https://github.com/actions/github-script/compare/v5...v6)

---
updated-dependencies:
- dependency-name: actions/github-script
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2022-04-19 09:29:29 -07:00
Naveen d2329f8cba chore: Included githubactions for dependabot (#6080)
This should help with keeping the GitHub actions updated on new releases. This will also help with keeping it secure.

Dependabot helps in keeping the supply chain secure https://docs.github.com/en/code-security/dependabot

GitHub actions up to date https://docs.github.com/en/code-security/dependabot/working-with-dependabot/keeping-your-actions-up-to-date-with-dependabot

https://github.com/ossf/scorecard/blob/main/docs/checks.md#dependency-update-tool

Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
2022-04-18 09:00:08 -07:00
Naveen e0d81ef03d chore: Set permissions for GitHub actions (#6077)
- Included permissions for the action. https://github.com/ossf/scorecard/blob/main/docs/checks.md#token-permissions

https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#permissions

https://docs.github.com/en/actions/using-jobs/assigning-permissions-to-jobs

[Keeping your GitHub Actions and workflows secure Part 1: Preventing pwn requests](https://securitylab.github.com/research/github-actions-preventing-pwn-requests/)

 Restrict the GitHub token permissions only to the required ones; this way, even if the attackers will succeed in compromising your workflow, they won’t be able to do much.

Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
2022-04-18 08:59:51 -07:00
alschmiedt 98d619b46e chore: update PR template to include formatting info (#5839) 2022-03-21 09:20:45 -07:00
Rachel Fenichel 423d2e58a1 chore: add release-please config on master (#5887) 2022-01-18 17:48:09 -08:00
Rachel Fenichel 4e87be7069 chore: clean up issue templates and pull request templates (#5827) 2021-12-15 14:53:26 -08:00
Rachel Fenichel 4a8668a358 release: merge master into develop (#5792)
* Bump ws from 7.4.4 to 7.5.1

Bumps [ws](https://github.com/websockets/ws) from 7.4.4 to 7.5.1.
- [Release notes](https://github.com/websockets/ws/releases)
- [Commits](https://github.com/websockets/ws/compare/7.4.4...7.5.1)

---
updated-dependencies:
- dependency-name: ws
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Bump lodash from 4.17.19 to 4.17.21

Bumps [lodash](https://github.com/lodash/lodash) from 4.17.19 to 4.17.21.
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](https://github.com/lodash/lodash/compare/4.17.19...4.17.21)

---
updated-dependencies:
- dependency-name: lodash
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Initial commit for appengine deploy action

* Update comments to be more descriptive

* Get deploy files from the correct directory

Previous path was `_deploy/`. New path is `../_deploy`.

* Create Github Action to comment on PR while develop is frozen (#5006)

* Create develop_freeze_comment.yml

* Update comments

* Fix typo and update uses

* Add test message

* Revert "Create Github Action to comment on PR while develop is frozen (#5006)" (#5013)

This reverts commit 8c635b5fbc.

* Revert "Get deploy files from the correct directory"

* Bump hosted-git-info from 2.8.4 to 2.8.9 (#4980)

Bumps [hosted-git-info](https://github.com/npm/hosted-git-info) from 2.8.4 to 2.8.9.
- [Release notes](https://github.com/npm/hosted-git-info/releases)
- [Changelog](https://github.com/npm/hosted-git-info/blob/v2.8.9/CHANGELOG.md)
- [Commits](https://github.com/npm/hosted-git-info/compare/v2.8.4...v2.8.9)

---
updated-dependencies:
- dependency-name: hosted-git-info
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Pull Request template for goog.module PRs

Provide a standard pull request template more suited to PRs doing
goog.module conversions.

There's no way to pick a non-default template when creating a PR
manually, but this one can be used by adding
"&?template=goog_module.md" to the
https://github.com/cpcallen/bugs/compare/ URL.

* Update goog_module.md with suggested fixes

* Fix typos.
* Reinstate "Proposed Changes" section.
* Mark text that should be edit with ALL CAPS.

* Bump normalize-url from 4.5.0 to 4.5.1

Bumps [normalize-url](https://github.com/sindresorhus/normalize-url) from 4.5.0 to 4.5.1.
- [Release notes](https://github.com/sindresorhus/normalize-url/releases)
- [Commits](https://github.com/sindresorhus/normalize-url/commits)

---
updated-dependencies:
- dependency-name: normalize-url
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Bump trim-newlines from 3.0.0 to 3.0.1

Bumps [trim-newlines](https://github.com/sindresorhus/trim-newlines) from 3.0.0 to 3.0.1.
- [Release notes](https://github.com/sindresorhus/trim-newlines/releases)
- [Commits](https://github.com/sindresorhus/trim-newlines/commits)

---
updated-dependencies:
- dependency-name: trim-newlines
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Bump path-parse from 1.0.6 to 1.0.7

Bumps [path-parse](https://github.com/jbgutierrez/path-parse) from 1.0.6 to 1.0.7.
- [Release notes](https://github.com/jbgutierrez/path-parse/releases)
- [Commits](https://github.com/jbgutierrez/path-parse/commits/v1.0.7)

---
updated-dependencies:
- dependency-name: path-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* Bump postcss from 7.0.35 to 7.0.36 (#4928)

Bumps [postcss](https://github.com/postcss/postcss) from 7.0.35 to 7.0.36.
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](https://github.com/postcss/postcss/compare/7.0.35...7.0.36)

---
updated-dependencies:
- dependency-name: postcss
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Rollup of all Q3 message changes (#5565)

Courtesy of Translatewiki

* Add Croatian to Code demo (#5583) (#5611)

Contributed courtesy of Lidija Kralj.

* chore: create release.yml (#5588)

* Create release.yml

* chore: update release.yml

* chore: update dependabot messages and labels (#5653)

The same as #5618, but against master because dependabot ignores changes on develop.

* chore: Create separate report_clang_format workflow in master branch (#5670)

The initial version of this workflow just uses `console.log` to report
the context of the `context` object.  It is hoped that there will
be enough information in this context to identify the PR to comment on,
without the `check_clang_format` workflow having to upload a
'build artefact' of some kind - see example of what I hoe to avoid
starting at `ReceivePR.yml` here:

https://securitylab.github.com/research/github-actions-preventing-pwn-requests/

A follow-up PR will (if possible) add the code to create comments
when `check_clang_format` fails.

Part of #5659.

* chore: Remove spurious extra `on` clause in report_clang_format.yml (#5671)

Removes an unwanted `on` clause that was overlooked in #5670 by author and reviewer.

* chore: More spelunking in GitHub Action data (#5673)

Looking for information about PR that triggered original
check_clang_format run.

* chore: revert github action (#5675)

* fix: Don't try to set text fields to null on cancel (#5692)

Mobile users get a window.prompt as an input, if they press the cancel button the return value is null.  Don't attempt to set the value of the field to null.

Caused errors in the custom note field which inherits from FieldTextInput.  Detected in Blockly Games Music.

This PR is for the master branch and includes a recompile.  The develop branch has changed enough that a cherrypick from develop to master won't work.  The bug in question represents a significant number of the errors being reported from Blockly Games.

* Update dependabot.yml (#5705)

* chore: run clang-format

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Monica Kozbial <6621618+moniika@users.noreply.github.com>
Co-authored-by: Christopher Allen <cpcallen+git@google.com>
Co-authored-by: Neil Fraser <fraser@google.com>
Co-authored-by: alschmiedt <aschmiedt@google.com>
2021-12-07 09:43:57 -08:00
alschmiedt 6448528e9a chore: applies fixes to clang format (#5677)
* chore: update the clang-format version

* chore: specify the clang format version to use

* chore: remove style tag since it will default to file
2021-11-05 12:55:09 -07:00
Christopher Allen f4381bb373 fix: Don't crash when unable to post comment to PR (#5669)
... by not ever posting such comments from this workflow.

There will be a separate PR for the other workflow that does post
comments, because it needs to be in the master branch.

Part of #5659.
2021-11-02 20:17:53 +00:00
Christopher Allen 8a60734071 chore: Update dependabot labelling config (#5664)
Label all dependabot PRs as "PR: dependencies", following
@rachel-fenichel's recent rationalisation of issue/PR labels.
2021-11-02 18:33:49 +00:00
Christopher Allen 7ff6b93eb5 chore: Fix permissions for Assign requested reviewers workflow (#5666)
* refactor: Inline assign_reviewers script to avoid checkout

Per https://securitylab.github.com/research/github-actions-preventing-pwn-requests/
it is not safe to do a checkout of the submitter-supplied code
AND THEN RUN IT (via require).  This is pretty bad.

We want to give this script more permissions by running it
`on: [pull_request_target]` (instead of `pull_request`); this would
give it permission to modify the PR (e.g. add comments, change
assignment).  While it would be OK to do a checkout with default
parameters (which in `pull_request_target` would check out *our*
branch rather than the submitted one) it simplest just to inline
this small script and thereby obviate the need to do a checkout at all.

* chore: Give assign_reviewers action required permissions

Changing it from `on: [pull_request]` to `on: [pull_request_review]`
will give the action write access to our repository, allowing it to
change the assignment of the PR.

This is now safe as the script does not ever check out any
submitter-supplied code.

* docs: Comment tweaks for assign_reviewers.yml
2021-11-02 15:55:02 +00:00
alschmiedt cfb995c073 chore: adds an action to assign reviewers from requested reviewers(#5642) 2021-10-29 15:44:33 -07:00
alschmiedt 8a89e080fd chore: adds a check for properly formatted files (#5624)
* chore: add check for clang format

* chore: updates clang format script
2021-10-25 11:43:59 -07:00
Rachel Fenichel 817a79745c chore: update dependabot messages and labels (#5618) 2021-10-18 17:27:45 -07:00
kozbial 6fc90213a6 Merge branch 'develop' into merge-develop-to-goog_module 2021-09-21 16:04:33 -07:00
alschmiedt 631b3ead5a chore: remove support for node 10 (#5496) 2021-09-17 13:34:56 -07:00
alschmiedt 0e43ddc4ba fix: adds a fix for npm install failing (#5471) 2021-09-15 13:12:02 -07:00
Monica Kozbial 7974ff5159 Add labels instead of setting in github action (#5324)
Update `tag_module_cleanup.yml` logic to add labels instead of setting labels list.

Fixes: https://github.com/google/blockly/issues/5298
2021-08-06 15:46:21 -07:00
Rachel Fenichel 1376f1e6c2 Revert "fix: update tag module workflow to add labels instead of setting one label" 2021-08-05 11:53:38 -07:00
Rachel Fenichel 08cdbcf8c7 fix: update tag module workflow to add labels instead of setting one label 2021-08-04 17:35:34 -07:00
Maribeth Bottorff e9a1668445 Only lint once in CI (#5128) 2021-07-20 16:48:30 -07:00
Aaron Dodson 78a5d98c20 Gzip build output files in update_metadata.sh instead of update_metadata.yml. 2021-07-20 16:48:30 -07:00
Aaron Dodson eddc5f676b Calculate metadata size from freshly-built and gzipped Blockly. 2021-07-20 16:48:30 -07:00
Aaron Dodson 8d3c39212b Added script and workflow to automatically update build artifact sizes in check_metadata.sh 2021-07-20 16:48:30 -07:00
Maribeth Bottorff 69dbcf17a2 Only lint once in CI (#5128) 2021-07-19 18:14:23 -07:00
Rachel Fenichel 8c0304ece8 Use pull request target trigger for tagging cleanup 2021-07-16 14:02:44 -07:00
Rachel Fenichel 05f8c27302 Use full commit hash. 2021-07-16 11:52:38 -07:00
Rachel Fenichel a9e60851ef Auto-tag and add milestones for cleanup PRs
For new pull requests against the goog_module branch, adds the 'type: cleanup'  label and sets the milestone to q3 2021 release.

Based on [this PR](https://github.com/eclipse-theia/theia/pull/8631/files).
2021-07-16 11:49:35 -07:00
Aaron Dodson 8d33869a4a Gzip build output files in update_metadata.sh instead of update_metadata.yml. 2021-07-14 10:58:08 -07:00
Aaron Dodson 1fd4b9bb21 Calculate metadata size from freshly-built and gzipped Blockly. 2021-07-13 08:43:52 -07:00
Aaron Dodson 80d0314ca2 Added script and workflow to automatically update build artifact sizes in check_metadata.sh 2021-07-12 13:37:25 -07:00
Monica Kozbial b2490c57b8 Create develop_freeze.yml (#5012)
* Create develop_freeze.yml

* Remove test message
2021-07-09 15:10:35 -07:00
Rachel Fenichel f5e4022407 Merge pull request #4998 from rachel-fenichel/deploy_appengine_workflow
Initial commit for appengine deploy action
2021-07-08 19:11:37 -07:00
Rachel Fenichel 16a11e419a Update comments to be more descriptive 2021-07-08 19:05:56 -07:00
Aaron Dodson fa95a3a332 Added @google/blockly-eng as the owner of the entire repo 2021-07-08 14:23:52 -07:00
Rachel Fenichel 23602bb177 Initial commit for appengine deploy action 2021-07-07 17:57:48 -07:00
Aaron Dodson 146840e09f Updated CI config to run new simplified test target. 2021-06-14 18:54:13 +00:00
Christopher Allen e65a396809 Revert removal of node v10 from build matrix. 2021-06-14 19:41:12 +01:00
Christopher Allen f94cb15021 Re-disable CI on macOS
I have been unable to resolve the mocha test issues, so leave macOS
disabled for the moment.
2021-06-14 19:41:12 +01:00
Christopher Allen 5cc0df775d Reenable CI on macOS
Working on fixing #2114.
2021-06-14 19:41:12 +01:00
Christopher Allen 403f3119a6 Bump node.js versions for CI
Remove v12 and add v16, per https://nodejs.org/en/about/releases/
2021-06-14 19:41:12 +01:00
alschmiedt be1dcb4b94 Updates dependabot to ignore jsdom changes (#4777) 2021-04-14 09:11:05 -07:00
Maribeth Bottorff 8548b54221 Merge branch 'master' into rc_2021_3 2021-03-25 09:45:49 -07:00
alschmiedt 26872cdcf6 Update pull request template (#4625) 2021-02-09 14:20:12 -08:00
alschmiedt 791ce2b7fb Migrate our tests to use github actions (#4614) (#4626) 2021-02-09 12:57:27 -08:00
alschmiedt 2dbf441e9b Trigger build on pull request (#4615) 2021-02-02 11:37:11 -08:00
alschmiedt 26beff4b2c Migrate our tests to use github actions (#4614) 2021-02-02 10:08:51 -08:00
Rachel Fenichel 9e5fa749d4 File naming 2020-08-13 16:36:44 -07:00