Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Document cylc lint #480

Merged
merged 4 commits into from
Jul 27, 2022
Merged

Document cylc lint #480

merged 4 commits into from
Jul 27, 2022

Conversation

wxtim
Copy link
Member

@wxtim wxtim commented Jun 16, 2022

Will only work with branch from cylc/cylc-flow#4900

Requirements check-list

  • I have read CONTRIBUTING.md and added my name as a Code Contributor.

@wxtim wxtim requested review from MetRonnie and datamel June 16, 2022 13:52
@wxtim wxtim self-assigned this Jun 16, 2022
@wxtim wxtim added this to the 8.0rc4 milestone Jun 16, 2022
@MetRonnie MetRonnie changed the title Document Cylc 728 Document cylc lint Jun 27, 2022
Copy link
Member

@MetRonnie MetRonnie left a comment

Choose a reason for hiding this comment

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

728 -> lint

@wxtim wxtim requested a review from MetRonnie June 28, 2022 09:36
@hjoliver hjoliver modified the milestones: 8.0rc4, 8.0.0 Jul 7, 2022
@wxtim wxtim mentioned this pull request Jul 11, 2022
7 tasks
Copy link
Contributor

@datamel datamel left a comment

Choose a reason for hiding this comment

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

Minor comment, you know our users level of knowledge better than me so I'll leave it up to you.

src/7-to-8/major-changes/config-changes.rst Outdated Show resolved Hide resolved
Co-authored-by: Ronnie Dutta <61982285+MetRonnie@users.noreply.github.com>
@MetRonnie
Copy link
Member

(Nudging tests)

@MetRonnie MetRonnie closed this Jul 27, 2022
@MetRonnie MetRonnie reopened this Jul 27, 2022
Copy link
Member

@MetRonnie MetRonnie left a comment

Choose a reason for hiding this comment

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

1 suggestion

src/7-to-8/major-changes/config-changes.rst Outdated Show resolved Hide resolved
Co-authored-by: Ronnie Dutta <61982285+MetRonnie@users.noreply.github.com>
@MetRonnie MetRonnie merged commit 8f59507 into cylc:master Jul 27, 2022
@MetRonnie MetRonnie added the content Addition or modification of documentation label Jul 27, 2022
@wxtim wxtim deleted the document-728-script branch July 27, 2022 12:59
wxtim added a commit to wxtim/cylc-doc that referenced this pull request Jul 29, 2022
…heck

* 'master' of github.com:cylc/cylc-doc:
  Update intersphinx mapping
  updated task-job.png (cylc#509)
  Document `cylc lint` (cylc#480)
  8.0.0 - release edits (cylc#507)
  document CYLC_WORKFLOW_NAME_BASE
  s/expected/required/g outputs. (cylc#492)
  clarify that run hosts should share a file system
  Update runahead limit documentation. (cylc#479)
  spelling corrections (cylc#500)
  Fix some shell variable names. (cylc#503)
  Document abort meaning.
  re-added summary of platforms to changes summary (cylc#487)
  Tweak new docs.
  Document how to continue a Cylc 7 run with Cylc 8.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Addition or modification of documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants