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

Add package doc for packages that currently lack it. #109

Merged
merged 2 commits into from
Jan 15, 2020

Conversation

negz
Copy link
Member

@negz negz commented Jan 15, 2020

Description of your changes

Checklist

I have:

  • Run make reviewable to ensure this PR is ready for review.
  • Ensured this PR contains a neat, self documenting set of commits.
  • Updated any relevant documentation, examples, or release notes.
  • Updated the RBAC permissions in clusterrole.yaml to include any new types.

Signed-off-by: Nic Cope <negz@rk0n.org>
@negz negz requested review from muvaf and hasheddan January 15, 2020 20:28
Copy link
Member

@hasheddan hasheddan left a comment

Choose a reason for hiding this comment

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

LGTM

@hasheddan
Copy link
Member

hmm I thought rerunning CI would fix this, but it looks like it failed again 🤔

@negz
Copy link
Member Author

negz commented Jan 15, 2020

Master is actually super broken at the moment. 🤔 I have no idea how #106 made it past CI, because it does not build.

...and the stuttering name linter warnings created by said fix.

Signed-off-by: Nic Cope <negz@rk0n.org>
@negz
Copy link
Member Author

negz commented Jan 15, 2020

@hasheddan PTAL - I've pushed a commit that fixes the package names.

@hasheddan hasheddan self-requested a review January 15, 2020 20:54
@upbound-bot
Copy link
Collaborator

90% (0.0%) vs master 90%

Copy link
Member

@hasheddan hasheddan left a comment

Choose a reason for hiding this comment

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

Looks like that did the trick 👍

@negz negz merged commit e0d566e into crossplane:master Jan 15, 2020
@negz negz deleted the pacdoc branch January 15, 2020 22:15
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.

3 participants