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.

Sign up for GitHub

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

Jump to bottom

Update dsl_inspec.md #5721

Merged
merged 1 commit into from
Nov 20, 2021
Merged

Update dsl_inspec.md #5721

merged 1 commit into from
Nov 20, 2021

Conversation

paulcalabro
Copy link
Contributor

Fixed a typo

@paulcalabro
Update dsl_inspec.md
eaf7646 
Fixed a typo
@paulcalabro paulcalabro requested review from a team as code owners November 11, 2021 20:19
@netlify
Copy link

netlify bot commented Nov 11, 2021 •
edited
Loading

✔️ Deploy Preview for chef-inspec ready!

🔨 Explore the source changes: eaf7646

🔍 Inspect the deploy log: https://app.netlify.com/sites/chef-inspec/deploys/618d7abc3793f60007835f53

😎 Browse the preview: https://deploy-preview-5721--chef-inspec.netlify.app/inspec/dsl_inspec

@github-actions github-actions bot added the Documentation ZH multi-repo label for the docs-team label Nov 11, 2021
@chef-expeditor
Copy link
Contributor

Hello paulcalabro! Thanks for the pull request!

Here is what will happen next:

  1. Your PR will be reviewed by the maintainers.
  2. Possible Outcomes
    a. If everything looks good, one of them will approve it, and your PR will be merged.
    b. The maintainer may request follow-on work (e.g. code fix, linting, etc). We would encourage you to address this work in 2-3 business days to keep the conversation going and to get your contribution in sooner.
    c. Cases exist where a PR is neither aligned to Chef InSpec's product roadmap, or something the team can own or maintain long-term. In these cases, the maintainer will provide justification and close out the PR.

Thank you for contributing!

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

kagarmoe
kagarmoe approved these changes Nov 19, 2021
View reviewed changes
Copy link

@kagarmoe kagarmoe left a comment

Choose a reason for hiding this comment

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

tenor-10423103

@kagarmoe
Copy link

Obvious fix - don't need DCO

@clintoncwolfe clintoncwolfe merged commit 8a3a698 into inspec:main Nov 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kagarmoe kagarmoe kagarmoe approved these changes

@clintoncwolfe clintoncwolfe Awaiting requested review from clintoncwolfe clintoncwolfe is a code owner automatically assigned from inspec/inspec-core-team

@Vasu1105 Vasu1105 Awaiting requested review from Vasu1105 Vasu1105 is a code owner automatically assigned from inspec/inspec-core-team

Assignees
No one assigned
Labels
Documentation ZH multi-repo label for the docs-team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@paulcalabro @kagarmoe @clintoncwolfe