Skip to content

Fix Neo VM target frameworks#2989

Merged
shargon merged 1 commit intoneo-project:masterfrom
lock9:fix-vm-targetframeworks
Nov 22, 2023
Merged

Fix Neo VM target frameworks#2989
shargon merged 1 commit intoneo-project:masterfrom
lock9:fix-vm-targetframeworks

Conversation

@lock9
Copy link
Contributor

@lock9 lock9 commented Nov 22, 2023

There were two problems that had to be fixed.

1 - The property 'TargetFramework' was replacing the 'TargetFrameworks' property on the Neo.VM project.
2 - The Neo.VM project is using the Lang 9.0 property that wasn't ported from the other repository.

How to reproduce the error:
On master, use dotnet pack -c Debug -o out --include-source. It will throw an error

How to verify:
Using this branch, try the same command again: dotnet pack -c Debug -o out --include-source. It should generate the files without errors (but a few warnings about 'package readmes')

This was referenced Nov 22, 2023
@shargon shargon merged commit 2e7e70e into neo-project:master Nov 22, 2023
Jim8y added a commit to Jim8y/neo that referenced this pull request Dec 31, 2023
* 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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants