samjtro / schwab

⚡ fast unofficial schwab traderapi hook
https://pkg.go.dev/github.com/samjtro/schwab
GNU General Public License v2.0
3 stars 0 forks source link

Continuous Integration for testing new commits #10

Closed samjtro closed 1 month ago

samjtro commented 1 year ago

I am building a Github Actions workflow to utilize credentials from a testing app on developer.tdameritrade.com in order to test new commits. The main reason for this is to ensure the integrity of new commits; if more people start working on this project, that will be a requirement. It will essentially just run the go tests I've already built out in the test folder with the credentials from the testing app to authenticate the requests, that way on every commit you are required to have tested (something not all people tend to do). It will also enable us to share the status of the repo in the Readme, in case a legitimate need for breaking the code has been introduced.

Development of this workflow is happening here: branch:testing-ci

Any help would be greatly appreciated! I have finished the go.yml for general testing (easy), but need to figure out the best way to store the tda-config.env file securely (might involve editing the utils.go file to enable placing the .env file in the repo itself).

samjtro commented 1 month ago

The schwab package's sub test is considerably behind current development, will get back to this eventually but the schwab migration has taken considerable priority.

Planning on a skeleton version of ci being implemented in v1.0.0 that we can improve upon.

samjtro commented 1 month ago

going to remove this as it isn't necessary. i can verify everything builds locally for now, and now that schwab has taken over this doesn't seem feasible.