-
Notifications
You must be signed in to change notification settings - Fork 7
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
chore: purge rewards and format #14
Conversation
@Keyrxng The build workflow is failing :( |
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks mostly fine just cspell related nits
.cspell.json
Outdated
], | ||
"dictionaries": ["typescript", "node", "software-terms", "html"], | ||
"import": ["@cspell/dict-typescript/cspell-ext.json", "@cspell/dict-node/cspell-ext.json", "@cspell/dict-software-terms"], | ||
"ignoreRegExpList": ["[0-9a-fA-F]{6}"] | ||
"ignoreRegExpList": ["[0-9a-fA-F]{6}"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is for colors? Where is the issue?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's just because I've added a comma it's registering a full line change but no dramas
@gitcoindev ready when you are |
Is this ready to be merged? |
@Keyrxng thank you for pinging me again, yes! |
I can never tell if it's rude of me to be pinging or if that should be the default lmao I get you folks are busy but cheers |
In my case: If I do not reply within 48hrs, do ping me -) |
Relates to ubiquity/.github#98
/rewards
yarn format
fixes