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

support no key columns in print method (make_row_df) #73 #74

Merged
merged 1 commit into from Dec 20, 2022

Conversation

gmbecker
Copy link
Contributor

No description provided.

Copy link
Contributor

@insights-engineering-bot insights-engineering-bot left a comment

Choose a reason for hiding this comment

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

LGTM 🤖

@github-actions
Copy link
Contributor

Unit Tests Summary

  1 files    3 suites   0s ⏱️
  8 tests   8 ✔️ 0 💤 0
48 runs  48 ✔️ 0 💤 0

Results for commit 83bf023.

@gmbecker gmbecker enabled auto-merge (squash) December 20, 2022 22:33
@gmbecker
Copy link
Contributor Author

@cicdguy What is going on here? Why did this not merge?

@gmbecker gmbecker merged commit c168dd7 into main Dec 20, 2022
@gmbecker gmbecker deleted the 73_no_keycols branch December 20, 2022 22:35
@cicdguy
Copy link
Contributor

cicdguy commented Dec 20, 2022

@cicdguy What is going on here? Why did this not merge?

What do you mean? Looks like it's merged

@gmbecker
Copy link
Contributor Author

I had to rerun the actions an then it immediately merged. It looked like there were two R CMD check actions required, one which was performed and one which was skipped.

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.

None yet

3 participants