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

feat: run pod lib lint during dryrun #1576

Merged
merged 1 commit into from
Oct 8, 2020

Conversation

hborawski
Copy link
Collaborator

@hborawski hborawski commented Oct 8, 2020

What Changed

Tap the beforeShipIt hook to run pod lib lint during a dryRun of shipit.

Why

During an actual shipit call, pod repo push is executed to push the updated podspec file to the specs repository. Internally, pod repo push will call pod lib lint to ensure the podspec file validates before pushing it.

During a dryrun, we will now run pod lib lint to simulate part of pod repo push while giving feedback to the user if shipit could have succeeded.

Todo:

  • Add tests
  • Add docs

Change Type

Indicate the type of change your pull request is:

  • documentation
  • patch
  • minor
  • major
📦 Published PR as canary version: under canary scope @auto-canary@9.55.1-canary.1576.19315.0

✨ Test out this PR locally via:

npm install @auto-canary/bot-list@9.55.1-canary.1576.19315.0
npm install @auto-canary/auto@9.55.1-canary.1576.19315.0
npm install @auto-canary/core@9.55.1-canary.1576.19315.0
npm install @auto-canary/all-contributors@9.55.1-canary.1576.19315.0
npm install @auto-canary/brew@9.55.1-canary.1576.19315.0
npm install @auto-canary/chrome@9.55.1-canary.1576.19315.0
npm install @auto-canary/cocoapods@9.55.1-canary.1576.19315.0
npm install @auto-canary/conventional-commits@9.55.1-canary.1576.19315.0
npm install @auto-canary/crates@9.55.1-canary.1576.19315.0
npm install @auto-canary/docker@9.55.1-canary.1576.19315.0
npm install @auto-canary/exec@9.55.1-canary.1576.19315.0
npm install @auto-canary/first-time-contributor@9.55.1-canary.1576.19315.0
npm install @auto-canary/gem@9.55.1-canary.1576.19315.0
npm install @auto-canary/gh-pages@9.55.1-canary.1576.19315.0
npm install @auto-canary/git-tag@9.55.1-canary.1576.19315.0
npm install @auto-canary/gradle@9.55.1-canary.1576.19315.0
npm install @auto-canary/jira@9.55.1-canary.1576.19315.0
npm install @auto-canary/maven@9.55.1-canary.1576.19315.0
npm install @auto-canary/npm@9.55.1-canary.1576.19315.0
npm install @auto-canary/omit-commits@9.55.1-canary.1576.19315.0
npm install @auto-canary/omit-release-notes@9.55.1-canary.1576.19315.0
npm install @auto-canary/pr-body-labels@9.55.1-canary.1576.19315.0
npm install @auto-canary/released@9.55.1-canary.1576.19315.0
npm install @auto-canary/s3@9.55.1-canary.1576.19315.0
npm install @auto-canary/slack@9.55.1-canary.1576.19315.0
npm install @auto-canary/twitter@9.55.1-canary.1576.19315.0
npm install @auto-canary/upload-assets@9.55.1-canary.1576.19315.0
# or 
yarn add @auto-canary/bot-list@9.55.1-canary.1576.19315.0
yarn add @auto-canary/auto@9.55.1-canary.1576.19315.0
yarn add @auto-canary/core@9.55.1-canary.1576.19315.0
yarn add @auto-canary/all-contributors@9.55.1-canary.1576.19315.0
yarn add @auto-canary/brew@9.55.1-canary.1576.19315.0
yarn add @auto-canary/chrome@9.55.1-canary.1576.19315.0
yarn add @auto-canary/cocoapods@9.55.1-canary.1576.19315.0
yarn add @auto-canary/conventional-commits@9.55.1-canary.1576.19315.0
yarn add @auto-canary/crates@9.55.1-canary.1576.19315.0
yarn add @auto-canary/docker@9.55.1-canary.1576.19315.0
yarn add @auto-canary/exec@9.55.1-canary.1576.19315.0
yarn add @auto-canary/first-time-contributor@9.55.1-canary.1576.19315.0
yarn add @auto-canary/gem@9.55.1-canary.1576.19315.0
yarn add @auto-canary/gh-pages@9.55.1-canary.1576.19315.0
yarn add @auto-canary/git-tag@9.55.1-canary.1576.19315.0
yarn add @auto-canary/gradle@9.55.1-canary.1576.19315.0
yarn add @auto-canary/jira@9.55.1-canary.1576.19315.0
yarn add @auto-canary/maven@9.55.1-canary.1576.19315.0
yarn add @auto-canary/npm@9.55.1-canary.1576.19315.0
yarn add @auto-canary/omit-commits@9.55.1-canary.1576.19315.0
yarn add @auto-canary/omit-release-notes@9.55.1-canary.1576.19315.0
yarn add @auto-canary/pr-body-labels@9.55.1-canary.1576.19315.0
yarn add @auto-canary/released@9.55.1-canary.1576.19315.0
yarn add @auto-canary/s3@9.55.1-canary.1576.19315.0
yarn add @auto-canary/slack@9.55.1-canary.1576.19315.0
yarn add @auto-canary/twitter@9.55.1-canary.1576.19315.0
yarn add @auto-canary/upload-assets@9.55.1-canary.1576.19315.0

@adierkens adierkens added hacktoberfest minor Increment the minor version when merged labels Oct 8, 2020
@hipstersmoothie hipstersmoothie merged commit c8b7a57 into intuit:master Oct 8, 2020
@adierkens
Copy link
Collaborator

🚀 PR was released in v9.56.0 🚀

@adierkens adierkens added the released This issue/pull request has been released. label Oct 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest minor Increment the minor version when merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants