Skip to content

Conversation

aadcg
Copy link

@aadcg aadcg commented Aug 19, 2022

No description provided.

@aartaka
Copy link

aartaka commented Aug 19, 2022

But then, maybe add more equality operators, to mirror the toolset of CL in general? Not much left: string-equal, char=, char-equal, =. And then /= and -not-equal versions of those...

@aadcg aadcg force-pushed the add-assert-string= branch from 93f8778 to b3ddcc1 Compare August 22, 2022 08:52
@aadcg
Copy link
Author

aadcg commented Aug 22, 2022

I don't see why not @aartaka.

PR revised accordingly.

Assertions for string=, string-equal, string/=, string-not-equal, char=,
char-equal, char/=, char-not-equal, = /=.
@aadcg aadcg force-pushed the add-assert-string= branch from b3ddcc1 to 12eb4f9 Compare August 22, 2022 09:07
@aartaka
Copy link

aartaka commented Aug 29, 2022

CC @bobbysmith007, @ryepup, how does that look to you?

@bobbysmith007
Copy link
Member

Its has been too long to have much insight, but it looks right to me.

@aartaka
Copy link

aartaka commented Aug 29, 2022

Care to merge then? :)

@bobbysmith007 bobbysmith007 merged commit 98dfa7c into AccelerationNet:master Aug 29, 2022
@aartaka
Copy link

aartaka commented Aug 30, 2022

Thanks!

@aadcg aadcg mentioned this pull request Sep 6, 2022
@aadcg aadcg deleted the add-assert-string= branch September 8, 2022 06:37
@Ambrevar
Copy link

There is a mistake in this pull request: none of the new symbols are exported.

@aadcg @bobbysmith007 ?

@aadcg
Copy link
Author

aadcg commented Sep 15, 2022

I will open a new PR. My mistake, sorry.

@aadcg
Copy link
Author

aadcg commented Sep 15, 2022

See #15.

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.

4 participants