Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
sannie-beep created a review on a pull request on AY2425S1-CS2103T-F08-2/tp
Yeah, I think conclusion: keep the original regex, and maybe we can tweak it to only alphabetical, no numeric. Let's see what the rest think too
wkxcass created a review comment on a pull request on AY2425S1-CS2103T-F08-2/tp
@sannie-beep ooh good point there, I'm actually fine with alphabetical! Can't seem to see a need for numbers.. may need others' inputs to decide as well~
wkxcass created a review comment on a pull request on AY2425S1-CS2103T-F08-2/tp
@k-hian As in, only the previous line with 3 `[\\p{Alnum}]`s can enforce correctly? Or the current one with 2 can as well XD
wkxcass created a review comment on a pull request on AY2425S1-CS2103T-F08-2/tp
So far yes, I also updated the one in `EditCommand` as well, just that it's under the `edit` PR (merged a few seconds ago)~
wkxcass pushed 3 commits to master AY2425S1-CS2103T-F08-2/tp
wkxcass closed an issue on AY2425S1-CS2103T-F08-2/tp
Alpha version - Message standardization (edit feature)
Tweak formats to ensure consistency with previously standardized messages.
wkxcass closed a pull request on AY2425S1-CS2103T-F08-2/tp
Alpha version - Message standardization (edit feature)
Fixes #99 Tweaked formats to ensure consistency with previously standardized messages.
k-hian opened a pull request on AY2425S1-CS2103T-F08-2/tp
AlphaVersion - Export puml as images
Fixes #104 #103
k-hian opened an issue on AY2425S1-CS2103T-F08-2/tp
PUML - Syntax error in BetterModelClassDiagram.puml
sannie-beep created a review comment on a pull request on AY2425S1-CS2103T-F08-2/tp
Nice! Do all our messages also follow this ? 🤔
sannie-beep created a review comment on a pull request on AY2425S1-CS2103T-F08-2/tp
Okay! I tested it, and it seems to work the same, so your edit seems good! Another thing is, do we want the name to be Alphanumerical, or just alphabetical?
k-hian created a review comment on a pull request on AY2425S1-CS2103T-F08-2/tp
The reason for the extra `[\\p{Alnum}]` is to enforce the "at least two word" rule. Without the extra portion, `Tan ` is an acceptable name.
k-hian created a review comment on a pull request on AY2425S1-CS2103T-F08-2/tp
Sorry, it seems like the change is made in the other PR. You can ignore this suggestion.
k-hian created a review comment on a pull request on AY2425S1-CS2103T-F08-2/tp
Could you standardise this language FindCommand (MESSAGE_NO_CRITERIA) as well?
k-hian created a review on a pull request on AY2425S1-CS2103T-F08-2/tp
LGTM, just the one extra message to standardise.
k-hian created a review on a pull request on AY2425S1-CS2103T-F08-2/tp
LGTM, just the one extra message to standardise.
wkxcass created a review comment on a pull request on AY2425S1-CS2103T-F08-2/tp
P.S. I went to experiment with this regex thingy, seems like this works fine as well? Not 100% certain if another `[\\p{Alnum}]` is necessary or redundant, open to feedback!
wkxcass created a review comment on a pull request on AY2425S1-CS2103T-F08-2/tp
Made use of imperative mood here #CS2101 🔥
wkxcass opened a pull request on AY2425S1-CS2103T-F08-2/tp
Alpha version - Code standardization (miscellaneous)
Fixes #101 - Tweak Javadoc comments to ensure alignment with current project. - Tweak formats to ensure consistency with previously standardized messages.
wkxcass opened an issue on AY2425S1-CS2103T-F08-2/tp
Alpha version - Code standardization (miscellaneous)
- Tweak Javadoc comments to ensure alignment with current project. - Tweak formats to ensure consistency with previously standardized messages.