Skip to content

Conversation

@erikmd
Copy link
Collaborator

@erikmd erikmd commented May 1, 2023

  • Kind: enhancement

Description

  • Ensure we use the latest release of the GHA docker/build-push-action.
  • and avoid warnings "Input 'repository' has been deprecated with message: v2 is now available through docker/build-push-action@v2".

Checklist

Note to maintainers

  • Read this wiki page.
  • Make sure the PR has a milestone.
  • Assign yourself before merging.
  • Either do a regular merge:
    • for PRs containing several commits following conventional-commits,
    • or for PRs containing 1 commit shared with a later PR (to preserve the SHA1)
  • Or do a squash-merge:
    • for PRs containing only 1 commit (not shared with a later PR),
    • or for PRs containing several commits that need not be kept in the history;
    • Update the commit message header with a conventional-commit type,
    • Add a footer Close #… if a related issue exists.

@erikmd erikmd added the kind: infrastructure CI, build tools, development tools label May 1, 2023
@erikmd erikmd self-assigned this May 1, 2023
@erikmd
Copy link
Collaborator Author

erikmd commented May 1, 2023

The CI is fine (albeit it's impossible to easily test the CD part before merging), so let's merge :'-)

@erikmd erikmd merged commit 69006f8 into ocaml-sf:master May 1, 2023
@erikmd
Copy link
Collaborator Author

erikmd commented May 1, 2023

We can note this new release of docker/build-push-action reduces build/push time from ~14' to ~10'30"

@erikmd
Copy link
Collaborator Author

erikmd commented May 1, 2023

There was a bug regarding the build_args: though. It should read: build-args:

https://github.com/ocaml-sf/learn-ocaml/actions/runs/4852619411

And the additional labels don't seem to be taken into account:

https://hub.docker.com/r/ocamlsf/learn-ocaml/tags

@erikmd erikmd deleted the bump-docker-gha branch May 1, 2023 16:20
@erikmd erikmd added this to the learn-ocaml 1.0.0 milestone May 16, 2023
erikmd added a commit to pfitaxel/learn-ocaml that referenced this pull request Jun 15, 2023
* master: (146 commits)
  chore(docker): Fix org.opencontainers.image.source label
  ci(docker): Replace `LABEL` Dockerfile commands with `labels:` (GHA) (ocaml-sf#551)
  fix(i18n): fix escaping issue in i18n
  fix(ui): update fr translation
  feat(ui): add some inline documentation to the teacher tab
  feat(ui): teacher tab: highlight the "apply" button on unsaved changes
  fix(ui): show different status for open and closed assigned exercises
  feat(ui): allow partial CSV export
  feat(ui): allow name input on teacher token creation
  feat(ui): better string input dialog
  chore: Add 2 checkboxes in PULL_REQUEST_TEMPLATE.md
  fix(web-app): Fix `process_html_file` w.r.t. `base_url`
  refactor(partition-view): Move adhoc CSS code to learnocaml_partition_view.css
  ci(docker): Fix build-args syntax (docker/build-push-action@v4)
  ci(docker): Fix GHA input name: s/build_args/build-args/
  ci(docker): Use docker/build-push-action@v4 (ocaml-sf#544)
  ci(macos): Run the `macOS` workflow as well in the weekly CI build
  feat(partition-view): Add a selector to show (tokens, nicks, or anon IDs) (ocaml-sf#540)
  feat(web-app): Add feedback button with internationalized tooltip
  feat(js_utils): Add HTMLElement.title support
  ...

 Conflicts:
	.ci-macosx.sh
	.github/workflows/static-builds.yml
	ci/docker-emacs-learn-ocaml-client/.emacs
	dune-project
	learn-ocaml-client.opam
	learn-ocaml.opam
	learn-ocaml.opam.locked
	scripts/static-build.sh
	src/app/learnocaml_common.ml
	src/app/learnocaml_common.mli
	src/app/learnocaml_description_main.ml
	src/app/learnocaml_index_main.ml
	src/app/learnocaml_teacher_tab.ml
	src/app/server_caller.ml
	src/main/dune
	src/main/learnocaml_client.ml
	src/main/learnocaml_server_main.ml
	src/main/linking_flags.sh
	src/server/learnocaml_server.ml
	src/state/learnocaml_api.ml
	src/state/learnocaml_api.mli
	src/utils/dune
	static/css/learnocaml_main.css
	static/index.html
	translations/fr.po
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind: infrastructure CI, build tools, development tools

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant