Skip to content

Conversation

@csouchet
Copy link
Contributor

@csouchet csouchet commented Sep 1, 2021

Closes #17

@csouchet csouchet added WIP 🚧 Pull request in progress and/or not ready for review. Used in addition to marking it as draft documentation Improvements or additions to documentation labels Sep 1, 2021
@csouchet csouchet mentioned this pull request Sep 1, 2021
2 tasks
@csouchet csouchet marked this pull request as ready for review September 1, 2021 15:00
@csouchet csouchet removed the WIP 🚧 Pull request in progress and/or not ready for review. Used in addition to marking it as draft label Sep 1, 2021
@csouchet csouchet requested a review from tbouffard September 1, 2021 15:00
- Ensure you are on the last tag locally: `git checkout vX.Y.Z`
- Open the project in the R Studio
- On the **Build** tab, click on **More** > **Build Binary Package**
<p align="center"><img title="Build Binary Package" src="./doc/r-studio-binary-build.png" alt="Build Binary Package"></p>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ we have to decide where to store the image used by the internal documentation. In #45, I put the readme image in a img folder, but we should only have one folder.
I don't know if doc is ok for R project, especially when we will implement #40. But for now, I am ok to use the doc folder.

@csouchet do we keep the doc folder? If so, I will update the other PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't have preferences. I just reproduce what we did on bpmn-visualization-js.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, let's start with the doc folder

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Integrated in #45 which has been merged in the main branch

@csouchet csouchet merged commit 50e8992 into main Sep 1, 2021
@csouchet csouchet deleted the 17-Document_the_release_process branch September 1, 2021 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[DOC] Document the release process

3 participants