Skip to content

feat: v2#52

Merged
bhumitattarde merged 108 commits into
mainfrom
v2
Jan 17, 2023
Merged

feat: v2#52
bhumitattarde merged 108 commits into
mainfrom
v2

Conversation

@bhumitattarde
Copy link
Copy Markdown
Collaborator

No description provided.

- suppress a lint warning
- renamed placeGTT to placeGtt
- moved gtt related structs to a separate file
- move mf related structs to separate file
- fix stray blank lines
- use DEFAULT* constants
- use camel case for structs and methods
- move structs to separate file
- use DEFAULT* constants
- convert magic numbers to constants
- move ws structs to separate file
- convert magic numbers to constants
- use DEFAULT* constants
- remove commented out code
Comment thread include/kitepp/kite/order.hpp Outdated
Comment thread include/kitepp/responses/market.hpp Outdated
Comment thread include/kitepp/responses/mf.hpp Outdated
Comment thread include/kitepp/responses/order.hpp Outdated
Comment thread include/kitepp/utils.hpp Outdated
Comment thread include/kitepp/kite/internal.hpp Outdated
Comment thread examples/example2.cpp
@sukesh-ak
Copy link
Copy Markdown

sukesh-ak commented Jan 2, 2023

Removing the previous comment.
Have an updated v2 fork with latest libraries, bug fixes and vcpkg manifest integration - details here #54

Comment thread docs/footer.html Outdated
@bhumitattarde bhumitattarde force-pushed the v2 branch 2 times, most recently from dae1fbf to a01ffea Compare January 11, 2023 16:56
Copy link
Copy Markdown
Member

@ajinasokan ajinasokan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

cc @rhnvrm @knadh

@bhumitattarde
Copy link
Copy Markdown
Collaborator Author

bhumitattarde commented Jan 12, 2023

@rhnvrm @ajinasokan I don't have any more changes to make. Can we merge this if there's nothing to add?

Comment thread .git-blame-ignore-revs
@@ -0,0 +1,2 @@
# change column limit to 80
39cd0478af477b63ecc925fa8af8b0c82e4566e6
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this file needed? What does this do?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

git blame ignores changes made by commits mentioned in this file. Mainly used to prevent git blame from getting trashed during mass-formatting commits.

@bhumitattarde bhumitattarde merged commit ed76af9 into main Jan 17, 2023
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