Skip to content

Commit 6dcc48d

Browse files
authored
[INFRA] Integrate feedback of the first release (#48)
Documentation for CRAN submission: simplify and remove screenshots Screenshots increase the source package size (we could ignore the files in RbuildIgnore) and are not useful: there is enough information provided by the linked contents. Source package content fixes - DESCRIPTION: title in title case - Content: add all md files and the doc directory The README.md file generates warnings because of unreachable links (contributing.md not in the package) Do not package the doc folder: only needed for the README.md file, non standard folder These changes will be reviewed when working on the documentation The GitHub worklow now builds the source package and run checks on it as done during the CRAN submission to detect errors earlier
1 parent a796864 commit 6dcc48d

File tree

6 files changed

+46
-12
lines changed

6 files changed

+46
-12
lines changed

.Rbuildignore

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,5 +3,6 @@
33
^.*\.iml$
44
^.*\.Rproj$
55
^\.Rproj\.user$
6-
^CONTRIBUTING\.md$
6+
^.*\.md$
7+
^doc$
78
^LICENSE$

.github/workflows/R-CMD-check.yaml

Lines changed: 32 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,37 @@ jobs:
8080

8181
- name: Upload check results
8282
if: failure()
83-
uses: actions/upload-artifact@main
83+
uses: actions/upload-artifact@v2
8484
with:
85-
name: ${{ matrix.config.os }}-r${{ matrix.config.r }}-results
85+
name: ${{ matrix.config.os }}-r${{ matrix.config.r }}-check-results
8686
path: check
87+
88+
# ######################################################################################################################
89+
# Package submission checks
90+
# https://cran.r-project.org/web/packages/policies.html#Submission
91+
# ######################################################################################################################
92+
- name: Build the source package
93+
run: |
94+
R CMD build .
95+
ls -la
96+
- name: Upload the source package
97+
uses: actions/upload-artifact@v2
98+
with:
99+
name: ${{ matrix.config.os }}-r${{ matrix.config.r }}-source-package
100+
path: bpmnVisualization_*.tar.gz
101+
102+
103+
# Sets up LaTeX with tinytex, for PDF generation
104+
# not needed before as the checks on the repo sources are done with --no-manual
105+
- uses: r-lib/actions/setup-tinytex@v1
106+
107+
- name: Check the source package
108+
id: source_package_checks
109+
run: R CMD check --as-cran bpmnVisualization_*.tar.gz
110+
111+
- name: Upload source package check results
112+
if: ${{ failure() && steps.source_package_checks.outcome == 'failure' }}
113+
uses: actions/upload-artifact@v2
114+
with:
115+
name: ${{ matrix.config.os }}-r${{ matrix.config.r }}-source-package-check-results
116+
path: bpmnVisualization.Rcheck

CONTRIBUTING.md

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -149,14 +149,17 @@ Create a new PR and merge it, if the version in the [DESCRIPTION](./DESCRIPTION)
149149

150150
- Pull the new version created by the release workflow: `git pull origin`
151151
- Ensure you are on the last tag locally: `git checkout vX.Y.Z`
152-
- Open the project in the R Studio
153-
- On the **Build** tab, click on **More** > **Build Binary Package**
154-
<p align="center"><img title="Build Binary Package" src="./doc/r-studio-binary-build.png" alt="Build Binary Package"></p>
155-
156-
- Go to the parent folder of the project. You should find a file named like `bpmnVisualization_X.Y.Z.tgz`.
157-
- Fill all the fields of this [web form](https://xmpalantir.wu.ac.at/cransubmit/), and load `bpmnVisualization_X.Y.Z.tgz` file.
158-
- Submit
159-
<p align="center"><img title="Submitting to CRAN" src="./doc/cran-submitting.png" alt="Submitting to CRAN"></p>
152+
- Build the source package. Please read [the CRAN Submission policies first](https://cran.r-project.org/web/packages/policies.html#Submission)
153+
- Solution 1: Build the source package with RStudio
154+
- Open the project in RStudio
155+
- On the **Build** tab, click on **More** > **Build Source Package**
156+
- Go to the parent folder of the project. You should find a file named like `bpmnVisualization_X.Y.Z.tgz`.
157+
- Solution 2.
158+
- from the project directory, run `R CMD build .
159+
- You should find a file named like `bpmnVisualization_X.Y.Z.tgz` in the project directory.
160+
- Do the actual submission
161+
- Fill all the fields of this [web form](https://xmpalantir.wu.ac.at/cransubmit/), and load `bpmnVisualization_X.Y.Z.tgz` file.
162+
- Submit
160163

161164
### Communicate about the release
162165

DESCRIPTION

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
Package: bpmnVisualization
22
Type: Package
3-
Title: Visualize process execution data on BPMN diagrams
3+
Title: Visualize Process Execution Data on BPMN Diagrams
44
Version: 0.1.0
55
Authors@R: c(person("Celine", "Souchet", role = c("aut", "cre"), email = "celine.souchet@bonitasoft.com"),
66
person("Thomas", "Bouffard", role = "aut"))

doc/cran-submitting.png

-62.1 KB
Binary file not shown.

doc/r-studio-binary-build.png

-62.1 KB
Binary file not shown.

0 commit comments

Comments
 (0)