search

lanl / BEE

Other
13 stars 3 forks source link

Documentation "typo" in Getting Started - Example Workflows section #761

Closed kchilleri closed 6 months ago

kchilleri commented 6 months ago

In the Getting Started - Example Workflows section under the CLAMR workflow examples (containerized application) portion, there are two instances stating to use bee config show to list configuration file contents. I believe this needs to be changed to beeflow config show instead.

pagrubel commented 6 months ago

@kchilleri Feel free to submit a PR, if you think you might have other documentation changes you can give it a WIP label until you want it merged.

kchilleri commented 6 months ago

@pagrubel For the PR, which two branches should I choose for comparing changes?

pagrubel commented 6 months ago

Start with develop and create a new branch for your PR, it can be helpful to name it with the issue such as issue761/fix- ... so it has some meaning. This will be a small PR most likely but it is helpful to be meaningful (WIP stands for Work in Progress and those PR's don't run the tests. Hope this helps.

pagrubel commented 6 months ago

Addressed and merged in #763