Skip to content

Conversation

@KushalMeghani1644
Copy link
Contributor

Closes #351

@KushalMeghani1644 KushalMeghani1644 requested a review from a team as a code owner October 16, 2025 10:53
@KushalMeghani1644
Copy link
Contributor Author

The 2 CI failing doesn't look like my fault, since the riscv-pac/CHANGELOG.md should be converted to riscv-types/CHANGELOG.md and the spell check is solved in another PR of mine #347 so solving it here might give a conflict, or maybe not.

@rmsyn
Copy link
Contributor

rmsyn commented Oct 16, 2025

Please stop spamming our project.

@rmsyn rmsyn closed this Oct 16, 2025
@KushalMeghani1644
Copy link
Contributor Author

KushalMeghani1644 commented Oct 17, 2025

Well this wasn't an attempt to "spam"! I guess we shall discuss things first! I was trying to resolve the issue romancardenas had opened, not trying to spam you, but if you look closely at the commits there were merge conflicts that ended up deleting my branches, I agree that I had opened 2PRs previously doing the EXACT same thing, but TBH this isn't a attempt to spam BUT a simple PR trying to resolve a issue. I definitely understand its stressing for y'all to see 10 thousand notifications on github especially since notifications on github aren't as good as other platforms (e.g. graphite.dev) that aside, I hope you got my intentions. AS A PROOF I can send screenshots of my terminal history trying to resolve conflicts that ended up deleting branches.

SCREENSHOTS of git reflog since tmux lost terminal history:

Screenshot_17-Oct_14-50-45_5559 Screenshot_17-Oct_14-51-13_8636

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.

Rename riscv-pac to riscv-types

2 participants