Skip to content

Exclude markdown files from 'Pester' folder #5636

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

Merged
merged 4 commits into from
Dec 6, 2017

Conversation

daxian-dbw
Copy link
Member

@daxian-dbw daxian-dbw commented Dec 6, 2017

PR Summary

Exclude markdown files from powershell.AppDir folder which is temporary for AppImage creation.
From https://travis-ci.org/PowerShell/PowerShell/jobs/312145407

The command "pwsh -File tools/travis.ps1" exited with 0.
1.82s$ if [[ "$TRAVIS_OS_NAME" == "linux" ]]; then mdspell '**/*.md' '!**/publish/**/*.md' --ignore-numbers --ignore-acronyms --report; fi
    powershell/powershell.AppDir/usr/bin/Modules/Pester/CHANGELOG.md
        2 | ed versions of that command. [GH-362] 

PR Checklist

Note: Please mark anything not applicable to this PR NA.

@TravisEz13
Copy link
Member

should be run with [package]

Copy link
Member

@TravisEz13 TravisEz13 left a comment

Choose a reason for hiding this comment

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

should be verified with a [package] commit

@daxian-dbw daxian-dbw changed the title Exclude markdown files from 'powershell.AppDir' folder Exclude markdown files from 'Pester' folder Dec 6, 2017
Copy link
Collaborator

@JamesWTruher JamesWTruher left a comment

Choose a reason for hiding this comment

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

LGTM

@daxian-dbw daxian-dbw merged commit 9a12349 into PowerShell:master Dec 6, 2017
@daxian-dbw daxian-dbw deleted the fixbuild branch December 6, 2017 20:11
TravisEz13 pushed a commit to TravisEz13/PowerShell that referenced this pull request Dec 7, 2017
@TravisEz13 TravisEz13 added this to the 6.0.0-RC.2 milestone Dec 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants