Skip to content

Commit fa6b522

Browse files
committed
Fix Build and push to GHCR step
1 parent 9637ef5 commit fa6b522

19 files changed

+517
-508
lines changed

β€Ž.Rhistory

Lines changed: 471 additions & 471 deletions
Large diffs are not rendered by default.

β€ŽNEWS.md

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,14 @@
1+
# rworkflows 1.0.1
2+
3+
## New features
4+
5+
- Change command "\nodocker" to "[nodocker]" for consistency with other commands.
6+
7+
## Bug fixes
8+
9+
- *action.yml*
10+
- `repository: ${{ github.repository }}` --> `repository: ${{ env.packageName }}`
11+
112
# rworkflows 1.0.0
213

314
## New features
@@ -15,6 +26,7 @@ Release versioning.
1526
the vignette is being rendered.
1627
* *use_vignette_docker*
1728
- Add *-autolink_bare_uris* bit to avoid CRAN check errors.
29+
* Change `\itemize` --> `describe` to avoid CRAN check errors.
1830

1931
# rworkflows 0.99.14
2032

β€ŽR/bioc_r_versions.R

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
#' \href{https://bioconductor.org/}{Bioconductor}.
55
#' @param bioc_version Version of Bioc to return info for.
66
#' Can be:
7-
#' \itemize{
7+
#' \describe{
88
#' \item{"devel"}{Get the current development version of Bioc.}
99
#' \item{"release"}{Get the current release version of Bioc.}
1010
#' \item{<numeric>}{A specific Bioc version number (e.g. \code{3.16}).}
@@ -13,7 +13,7 @@
1313
#' @param depth How many levels deep into the R version to include.
1414
#' For example, is the R version number is "4.2.0",
1515
#' the following depths would return:
16-
#' \itemize{
16+
#' \describe{
1717
#' \item{\code{depth=NULL}: }{"4.2.0"}
1818
#' \item{\code{depth=1}: }{"4"}
1919
#' \item{\code{depth=2}: }{"4.2"}

β€ŽR/check_cont.R

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
#'
33
#' Check that a list of containers are valid.
44
#' @inheritParams construct_runners
5-
#' @param domain Container repository API domain.
65
#' @returns Null
76
#'
87
#' @keywords internal

β€ŽR/construct_cont.R

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
#' (and the tag, if specified) exist using \link[rworkflows]{check_cont}.
55
#' @param default_registry The default container registry to use.
66
#' Options include:
7-
#' \itemize{
7+
#' \describe{
88
#' \item{"ghcr.io/" : }{\href{https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-container-registry}{GitHub Container Registry}}
99
#' \item{"docker.io/" : }{\href{https://hub.docker.com/}{DockerHub}}
1010
#' \item{"mcr.microsoft.com/" : }{\href{https://github.com/microsoft/ContainerRegistry}{Microsoft Container Registry}}

β€ŽR/fill_description.R

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
#' R package template.
77
#' For any given field, set its corresponding argument as follows
88
#' to get certain behaviour:
9-
#' \itemize{
9+
#' \describe{
1010
#' \item{\code{NULL}: }{Keeps the current value.}
1111
#' \item{\code{NA}: }{Removes the field from the
1212
#' \emph{DESCRIPTION} file entirely.}

β€ŽR/use_workflow.R

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
#' \href{https://github.com/neurogenomics/rworkflows}{rworkflows}
55
#' \href{https://github.com/features/actions}{GitHub Actions (GHA)}
66
#' @param template Workflow template name.
7-
#' \itemize{
7+
#' \describe{
88
#' \item{"rworkflows"}{A short workflow script that calls
99
#' the GitHub action from the GitHub Marketplace.
1010
#' The action is continually updated
@@ -71,10 +71,10 @@
7171
#' @param run_docker Whether to build and push a Docker container to DockerHub.
7272
#' @param docker_registry Docker container registry to push to.
7373
#' Options include:
74-
#' \itemize{
75-
#' \item{"ghcr.io"} : \href{https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-container-registry}{
76-
#' GitHub Container Registry}
77-
#' \item{"docker.io"} : \href{https://hub.docker.com/}{DockerHub}
74+
#' \describe{
75+
#' \item{"ghcr.io" : }{\href{https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-container-registry}{
76+
#' GitHub Container Registry}}
77+
#' \item{"docker.io" : }{\href{https://hub.docker.com/}{DockerHub}}
7878
#' }
7979
#' @param docker_user Docker registry username.
8080
#' Not used when \code{docker_registry="ghcr.io"}.

β€ŽREADME.Rmd

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ and/or [**Bioconductor**](https://bioconductor.org/) checks).
5050
to [**Codecov**](https://about.codecov.io/).
5151
9. πŸš€ (Re)builds and launches a documentation website for your R package.
5252
10. 🐳 Pushes a [**Docker**](https://www.docker.com/) container (with [**Rstudio**](https://posit.co/) and all dependencies pre-installed) to
53-
your choice of container registry (e.g. [GitHub Container Registry](https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-container-registry), [**DockerHub**](https://hub.docker.com/)).
53+
your choice of container registry (e.g. [**GitHub Container Registry**](https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-container-registry), [**DockerHub**](https://hub.docker.com/)).
5454
11. πŸ”­ Generates [workflow telemetry](https://github.com/runforesight/workflow-telemetry-action) report.
5555
12. πŸŽ– Updates relevant badges added to your README
5656
with `rworkflows::use_badges()`.

β€ŽREADME.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ status](https://github.com/neurogenomics/rworkflows/workflows/rworkflows_dev/bad
2424
Authors: <i>Brian Schilder, Alan Murphy, Nathan Skene</i>
2525
</h4>
2626
<h4>
27-
README updated: <i>Nov-01-2023</i>
27+
README updated: <i>Nov-02-2023</i>
2828
</h4>
2929

3030
## Intro
@@ -79,8 +79,8 @@ options to enable/disable/modify each step):
7979
package.
8080
10. 🐳 Pushes a [**Docker**](https://www.docker.com/) container (with
8181
[**Rstudio**](https://posit.co/) and all dependencies pre-installed)
82-
to your choice of container registry (e.g.Β [GitHub Container
83-
Registry](https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-container-registry),
82+
to your choice of container registry (e.g.Β [**GitHub Container
83+
Registry**](https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-container-registry),
8484
[**DockerHub**](https://hub.docker.com/)).
8585
11. πŸ”­ Generates [workflow
8686
telemetry](https://github.com/runforesight/workflow-telemetry-action)

β€Žaction.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -502,7 +502,7 @@ runs:
502502
## DockerHub steps
503503
- name: 🐳✏️ Create Docker file
504504
if: |
505-
(!contains(github.event.head_commit.message, '/nodocker')) && inputs.run_docker == 'true' && runner.os == 'Linux'
505+
(!contains(github.event.head_commit.message, '[nodocker]')) && inputs.run_docker == 'true' && runner.os == 'Linux'
506506
env:
507507
BASE_IMAGE: ${{ matrix.config.cont }}
508508
GITHUB_TOKEN: ${{ inputs.GITHUB_TOKEN }}
@@ -514,15 +514,15 @@ runs:
514514

515515
- name: πŸ³πŸš€ Build and push to GHCR
516516
if: |
517-
(!contains(github.event.head_commit.message, '/nodocker')) && inputs.run_docker == 'true' && runner.os == 'Linux' && inputs.docker_registry == 'ghcr.io'
517+
(!contains(github.event.head_commit.message, '[nodocker]')) && inputs.run_docker == 'true' && runner.os == 'Linux' && inputs.docker_registry == 'ghcr.io'
518518
uses: docker/build-push-action@v1
519519
env:
520520
GITHUB_TOKEN: ${{ inputs.GITHUB_TOKEN }}
521521
with:
522522
username: ${{ github.repository_owner }}
523523
password: ${{ inputs.GITHUB_TOKEN }}
524524
registry: ${{ inputs.docker_registry }}
525-
repository: ${{ github.repository }}
525+
repository: ${{ github.repository_owner }}/${{ env.packageName }}
526526
tag_with_ref: true
527527
tag_with_sha: false
528528
tags: |
@@ -534,7 +534,7 @@ runs:
534534
535535
- name: πŸ³πŸš€ Build and push to DockerHub
536536
if: |
537-
(!contains(github.event.head_commit.message, '/nodocker')) && inputs.run_docker == 'true' && runner.os == 'Linux' && inputs.docker_registry == 'docker.io'
537+
(!contains(github.event.head_commit.message, '[nodocker]')) && inputs.run_docker == 'true' && runner.os == 'Linux' && inputs.docker_registry == 'docker.io'
538538
uses: docker/build-push-action@v1
539539
env:
540540
GITHUB_TOKEN: ${{ inputs.GITHUB_TOKEN }}

0 commit comments

Comments
Β (0)