BHoM / Python_Toolkit

GNU Lesser General Public License v3.0
4 stars 2 forks source link

Add python code to nuspec as content files #148

Closed Tom-Kingstone closed 1 day ago

Tom-Kingstone commented 1 day ago

NOTE: Depends on

Issues addressed by this PR

Closes #146

See the nuspec file for LadybugTools_Toolkit as an example and compare to changes made here. As python bhom analytics now exists in Python_Toolkit code, this must be included in the nuspec content files as well. (not to mention future additions to Python_Toolkit python code)

Test files

Likely will have to merge this then see if the bot produces nugets that contain the python files first

Changelog

Additional comments

Tom-Kingstone commented 1 day ago

@BHoMBot check required

bhombot-ci[bot] commented 1 day ago
@Tom-Kingstone to confirm, the following actions are now queued: - check `code-compliance` - check `documentation-compliance` - check `project-compliance` - check `core` - check `null-handling` - check `serialisation` - check `versioning` - check `installer`
Tom-Kingstone commented 1 day ago

@BHoMBot check copyright-compliance

bhombot-ci[bot] commented 1 day ago
@Tom-Kingstone to confirm, the following actions are now queued: - check `copyright-compliance` There are 9 requests in the queue ahead of you.
Tom-Kingstone commented 1 day ago

@BHoMBot check installer

bhombot-ci[bot] commented 1 day ago
@Tom-Kingstone to confirm, the following actions are now queued: - check `installer` There are 32 requests in the queue ahead of you.
Tom-Kingstone commented 1 day ago

@BHoMBot check ready-to-merge

bhombot-ci[bot] commented 1 day ago
@Tom-Kingstone to confirm, the following actions are now queued: - check `ready-to-merge` There are 36 requests in the queue ahead of you.
bhombot-ci[bot] commented 1 day ago
The check `installer` has already been run previously and recorded as a successful check. This check has not been run again at this time.
Tom-Kingstone commented 1 day ago

@BHoMBot check ready-to-merge

bhombot-ci[bot] commented 1 day ago
@Tom-Kingstone to confirm, the following actions are now queued: - check `ready-to-merge` There are 57 requests in the queue ahead of you.
jamesramsden-bh commented 1 day ago

@BHoMBot check ready-to-merge

bhombot-ci[bot] commented 1 day ago
@jamesramsden-bh to confirm, the following actions are now queued: - check `ready-to-merge` There are 51 requests in the queue ahead of you.