Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

packaging update #279

Merged
merged 8 commits into from
May 9, 2024
Merged

packaging update #279

merged 8 commits into from
May 9, 2024

Conversation

jacalata
Copy link
Contributor

Building with pyinstaller v6 caused the executable to trigger automated malware alerts. Changing back to build with v5.13 gets an almost clean scan.

tabcmd 0.13 from github built with pyinstaller 6.x: ~30 vendors flagged it
https://www.virustotal.com/gui/file/829b092e47228c0cea2c54e0f75e63501b5e12135a76fef2dcbaba2deac228ed

my freshly built copy of 0.13 with pyinstaller 5.13: 2 vendors flagged it
https://www.virustotal.com/gui/file/b1c8be24ade44a8eae93e6a03d56fcd99caf389549c647fcb133b965d1c8e1f4/detection

tabcmd0.11.exe from github: was also built with 5.13: 1 vendor flaggged it
https://www.virustotal.com/gui/file/f6453b2549b9f696701e76d83b7c349a8710d3492a4fabf3029a6634c7f236f2

contributing.md Outdated
####To work with tabcmd, you need to have **Python 3.7+** installed.
## Install Tabcmd
> [!NOTE]
> These instructions are for people who want to write code in or using tabcmd. If > you are interested in tabcmd but not the code, see [here](Readme.md).
Copy link
Contributor

@anyoung-tableau anyoung-tableau May 6, 2024

Choose a reason for hiding this comment

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

I'm having some trouble understanding this line.

Also, I think there's a stray > in the 2nd sentence.

jacalata and others added 2 commits May 8, 2024 14:36
Co-authored-by: Andy Young <90729701+anyoung-tableau@users.noreply.github.com>
@jacalata jacalata merged commit 19aabf5 into main May 9, 2024
15 checks passed
@jacalata jacalata deleted the jac/windows-spec-update branch January 3, 2025 04:31
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.

3 participants