Skip to content

Conversation

@jimrothstein
Copy link

PLACEHOLDER - no review

@ddsjoberg ddsjoberg marked this pull request as draft June 27, 2025 19:52
@jimrothstein
Copy link
Author

@ddsjoberg
A draft ... If ok, I will put into proper style, roxygen, lint,
thx!

@jimrothstein jimrothstein marked this pull request as ready for review July 16, 2025 04:41
@ddsjoberg
Copy link
Collaborator

@jimrothstein the update looks good to me!

But it looks like we didn't pull the most recent changes into your branch before making the update. Can you resolve the merge conflicts?

If it were me, I would probably undo the updates I made, pull the updates from the main branch, then re-update the labels.

@jimrothstein
Copy link
Author

jimrothstein commented Jul 21, 2025 via email

@jimrothstein
Copy link
Author

@ddsjoberg
Propose to do this: (many ways to do; but also many wrong ways)

  • stash or just save the one *.R file that does the work.
  • current PR: close or delete
  • main: get all the updates
  • checkout a new branch, from main Call it temp
  • rebase my current work (ie outdated) onto the new temp branch.
  • push; create new PR

Worst case: copy/paste the R file onto temp branch; push, PR etc.

@ddsjoberg
Copy link
Collaborator

Instead of doing any fancy git work, I would just make a new branch that has all the updates from main, and rerun my script to update the labels. (I am all about avoiding complex git workflows!)

@jimrothstein
Copy link
Author

Please SEE VERSION2, PR #72

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.

2 participants