matrix-org / matrix-org/matrix-hookshot
unify (set)personaltoken syntax
- Dominant language
- TypeScript
- Stars
- 450
- Forks
- 95
- Avg merge
- 1d 2h
- Merged PRs (30d)
- 23
Description
https://github.com/Half-Shot/matrix-hookshot/blob/0c4b911a705f464d0a1edb51b2181845c4894b14/src/AdminRoom.ts#L350
https://github.com/Half-Shot/matrix-hookshot/blob/6d7112ae076b3b9befc899af788f0e876008381f/src/Github/AdminCommands.ts#L41
I haven't checked yet whether there are more accurences of this, but it's probably a good idea to use the same style
Contributor guide
Research direction
Start with the referenced command handling in src/AdminRoom.ts around line 350 and src/Github/AdminCommands.ts around line 41. Search the repository for other occurrences of the setpersonaltoken command, determine the consistent syntax from the existing usages, and finish when all occurrences follow the same style.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100