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

move neo-core projects into a dependency folder to make it clear. #871

Merged
merged 1 commit into from
Jan 19, 2024

Conversation

Jim8y
Copy link
Contributor

@Jim8y Jim8y commented Jan 17, 2024

Just find it misleading to mix everything in the src folder. so move the neo-core to a dependency folder. No logic is changed

@vncoelho
Copy link
Member

Why not move to neo-core the devpack instead?

I think we should archive here as we discussed in the examples PR.

@Jim8y
Copy link
Contributor Author

Jim8y commented Jan 18, 2024

@vncoelho

  1. Adding submodule is much faster than merging into monorrpo, i need it to debug the devpack, can't wait for months.

  2. Monorepo is for the ease of maintaining and debugging, now the submodule also work.

  3. A problem with the monorepo is that it needs to release files, might be too big since nccs also need to release files.

  4. Community members have asked to keep the devpack separate.

@Jim8y Jim8y merged commit 768f62c into neo-project:master Jan 19, 2024
1 check passed
@Jim8y Jim8y deleted the dependency branch January 19, 2024 02:41
Jim8y added a commit to Jim8y/neo-devpack-dotnet that referenced this pull request Jan 31, 2024
* master:
  [Framework: Add] Define Standard Enum (neo-project#877)
  Fix: Fix dependency (neo-project#876)
  create seperate test contracts (neo-project#868)
  fix-nep11token (neo-project#874)
  add comments to statements (neo-project#875)
  move neo-core projects into a dependency folder to make it clear. (neo-project#871)
Jim8y added a commit to Jim8y/neo-devpack-dotnet that referenced this pull request Feb 10, 2024
* master: (28 commits)
  Reduce notifications (neo-project#888)
  Core optimize optimizer (neo-project#887)
  print-stack (neo-project#873)
  ZipArchive use utf8; simplify GetOperandString
  [Compiler] move expressions out of convert file. (neo-project#882)
  misc refactor
  Zip and Unzip methods for files only
  add optimize test
  add debug notify into contract (neo-project#872)
  [Compiler: Add]verify the NEP11 and NEP17 standards while compile them. (neo-project#881)
  [Compiler: Comment] add comment to initial value (neo-project#880)
  refactor
  move to folder `Optimizer`
  optimizer to remove unused instructions
  [Framework: Add] Define Standard Enum (neo-project#877)
  Fix: Fix dependency (neo-project#876)
  create seperate test contracts (neo-project#868)
  fix-nep11token (neo-project#874)
  add comments to statements (neo-project#875)
  move neo-core projects into a dependency folder to make it clear. (neo-project#871)
  ...
Jim8y added a commit to Jim8y/neo-devpack-dotnet that referenced this pull request Feb 11, 2024
* master: (28 commits)
  Reduce notifications (neo-project#888)
  Core optimize optimizer (neo-project#887)
  print-stack (neo-project#873)
  ZipArchive use utf8; simplify GetOperandString
  [Compiler] move expressions out of convert file. (neo-project#882)
  misc refactor
  Zip and Unzip methods for files only
  add optimize test
  add debug notify into contract (neo-project#872)
  [Compiler: Add]verify the NEP11 and NEP17 standards while compile them. (neo-project#881)
  [Compiler: Comment] add comment to initial value (neo-project#880)
  refactor
  move to folder `Optimizer`
  optimizer to remove unused instructions
  [Framework: Add] Define Standard Enum (neo-project#877)
  Fix: Fix dependency (neo-project#876)
  create seperate test contracts (neo-project#868)
  fix-nep11token (neo-project#874)
  add comments to statements (neo-project#875)
  move neo-core projects into a dependency folder to make it clear. (neo-project#871)
  ...

# Conflicts:
#	src/Neo.Compiler.CSharp/MethodConvert.cs
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