Develop >>> Main #120
Merged
Develop >>> Main #120
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* factor: update analysis naming screen copy * factor: update analysis naming screen copy style
…113 (#118) * refactor: separate building exe and creating new release * upload artifacts only if triggered from release * debug: handle workflow inputs * remove precommit test * test windows exe run on PR * debug: fix executable filepath * debug: fix filepath to updated name * debug: path / to \ * refactor: implement run checks for macos * debug: is_release == false instead of !
* feat: 🎸 basic excel importer This Excel importer is very rudimentary but it gets the job done in most cases. It assumes that the data to be imported lives in one of the sheets in a "reasonable" position (i.e. headers in Row 1 and first column is A1). If a file has multiple sheet, the user can change the sheet to import. No explicit range selection in the sheet itself is possible. * run isort for styling --------- Co-authored-by: DeanEby <ebyd21@gmail.com>
* factor: remove initial output analysis * refactor: context dependent post-analysis menu * remove debug print statement * refactor: slight copy changes * debug: remove export outputs debug print
* test: 💍 expose CsvConfig for CsvTestData The CsvConfig class encodes how csv files can be loaded for testing, but it was never exposed or wired up in the CsvTestData file. This commit corrects this, and adds an example of how to use it to customize the way test CSV files are loaded for testing. * style: 💄 isort && black
VEDA95
approved these changes
Apr 8, 2025
Collaborator
|
Everything looks good. You're good to go to merge |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I tested the changes locally and everything seems to be in working order. Ben wants to create a new release so we can share our most recent additions.