Skip to content

Nuget Package Icon & Symbols#3020

Merged
shargon merged 5 commits intoneo-project:masterfrom
cschuchardt88:Package-icons
Dec 12, 2023
Merged

Nuget Package Icon & Symbols#3020
shargon merged 5 commits intoneo-project:masterfrom
cschuchardt88:Package-icons

Conversation

@cschuchardt88
Copy link
Member

@cschuchardt88 cschuchardt88 commented Dec 9, 2023 •
edited
Loading

Change Log

  • Added package icon to nuget package
  • Added snupkg symbols to nuget package

NOTE: nuget and myget require snupkg symbol format now.

Icon

image

Also you want to change your workflow. Because you are packaging up neo-cli, neo-gui and others that don't need packaging for myget and nuget; you publish the files as well.

@cschuchardt88 cschuchardt88 marked this pull request as draft December 10, 2023 00:43
@cschuchardt88 cschuchardt88 changed the title Nuget Package Icon & Symbols Update Workflow Dec 10, 2023
@cschuchardt88 cschuchardt88 changed the title Update Workflow Nuget Package Icon & Symbols Dec 10, 2023
@cschuchardt88 cschuchardt88 marked this pull request as ready for review December 10, 2023 01:42
@shargon
Copy link
Member

shargon commented Dec 10, 2023

Yes, could you remove cli and gui in nuget package?

shargon
shargon reviewed Dec 11, 2023
View reviewed changes
shargon
shargon reviewed Dec 11, 2023
View reviewed changes
.github/workflows/main.yml
- name : Pack (Neo)
run: |
dotnet pack ./src/Neo \
--configuration Debug \
Copy link
Member

@shargon shargon Dec 11, 2023

Choose a reason for hiding this comment

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

--include-source ?

Copy link
Member Author

@cschuchardt88 cschuchardt88 Dec 11, 2023

Choose a reason for hiding this comment

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

what does that do? Symbols? if so they get automatically generated on published.

Copy link
Member

@shargon shargon Dec 11, 2023

Choose a reason for hiding this comment

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

--include-symbols - it creates the symbols package.
--include-source - it creates the symbols package with a src folder inside containing the source files.

Copy link
Member Author

@cschuchardt88 cschuchardt88 Dec 11, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

Oh ya we can do that.

Copy link
Member Author

@cschuchardt88 cschuchardt88 Dec 11, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

This won't work see NuGet/Home#8589, spec doesnt allow it anymore, cause of the new format.

@shargon shargon merged commit 52f660a into neo-project:master Dec 12, 2023
@shargon
Copy link
Member

shargon commented Dec 12, 2023

@cschuchardt88 error : 'CI01440' is not a valid version string. (Parameter 'value')

@cschuchardt88
Copy link
Member Author

cschuchardt88 commented Dec 12, 2023

ok let me look

@cschuchardt88
Copy link
Member Author

cschuchardt88 commented Dec 12, 2023

its doing same as before git rev-list --count HEAD |xargs printf "CI%05d" |xargs dotnet pack -c Debug -o out --include-source --version-suffix

@cschuchardt88
Copy link
Member Author

cschuchardt88 commented Dec 12, 2023

@shargon
image

@cschuchardt88
Copy link
Member Author

cschuchardt88 commented Dec 12, 2023

Oh you know what it is, the Directory.Build.props isn't being used. I think its a bug with, cause it should be using parent directory. Ill fix, had problem with this on neo-express

Jim8y added a commit to Jim8y/neo that referenced this pull request Dec 31, 2023
@Jim8y
Merge branch 'master' into parallel-transaction-execution
f8f9ef2 
* master: (30 commits)
  Set project as nullable (neo-project#3042)
  Fix: fix equal (neo-project#3028)
  Added README to packages (neo-project#3026)
  Nuget MyGet Fix (neo-project#3031)
  Add: print out the stack (neo-project#3033)
  fixed myget (neo-project#3029)
  Fixed MyGet Workflow (neo-project#3027)
  Package icons - hotfix (neo-project#3022)
  Nuget Package Icon & Symbols (neo-project#3020)
  Fix warning (neo-project#3021)
  Neo-node Migration (neo-project#2990)
  Remove unnecessary default seedlist (neo-project#2980)
  Fix Neo VM target frameworks (neo-project#2989)
  Update Neo.VM location in README.md (neo-project#2988)
  Migrating Neo VM (neo-project#2970)
  3.6.2 (neo-project#2962)
  fix ut (neo-project#2959)
  Validate serialization during Contract deploy and Update (neo-project#2948)
  code optimization (neo-project#2958)
  check null scriptcontainer (neo-project#2953)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shargon shargon shargon approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cschuchardt88 @shargon