Skip to content

chore: migrate from setup tools to hatchling#57

Open
Jo-Byr wants to merge 1 commit into
masterfrom
hatchling
Open

chore: migrate from setup tools to hatchling#57
Jo-Byr wants to merge 1 commit into
masterfrom
hatchling

Conversation

@Jo-Byr
Copy link
Copy Markdown
Contributor

@Jo-Byr Jo-Byr commented May 18, 2026

No description provided.

@Jo-Byr Jo-Byr requested a review from jourdain May 18, 2026 13:54
@Jo-Byr Jo-Byr self-assigned this May 18, 2026
Comment thread src/trame_client/LICENSE Outdated
Comment thread MANIFEST.in Outdated
Comment thread pyproject.toml
Comment thread pyproject.toml Outdated
@jourdain
Copy link
Copy Markdown
Collaborator

Something to keep in mind. The conda recipe will break on next version because of hatchling switch.

Comment thread pyproject.toml
"src/trame",
"src/trame_client/module/user_provided_scripts/es/*",
"src/trame_client/module/user_provided_scripts/umd/*",
]
Copy link
Copy Markdown
Collaborator

@jourdain jourdain May 20, 2026

Choose a reason for hiding this comment

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

The packages should only be src/trame and src/trame_client

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Should I do like in trame-vtklocal and use

src/trame/**/*.py,
src/trame_client/**/*.py

?

Comment thread MANIFEST.in
recursive-exclude trame_client/module/vue2-www *.map
recursive-exclude trame_client/module/vue3-www *.map
include trame_client/LICENSE
graft src/trame_client/module/vue2-www
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

That full file should be removed

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'm not talking about the lines in the file, but the MANIFEST.in file all together

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.

2 participants