Commit Graph

159 Commits

Author SHA1 Message Date
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
Rachel Fenichel
9e29c06e4c Create .config.yml 2020-08-13 15:07:55 -07:00
Rachel Fenichel
a1075a8b83 Create dependabot.yml 2020-08-07 13:11:30 -07:00