Skip to content

Refactored repo layout#6

Closed
cschuchardt88 wants to merge 1 commit into
Hecate2:masterfrom
cschuchardt88:master
Closed

Refactored repo layout#6
cschuchardt88 wants to merge 1 commit into
Hecate2:masterfrom
cschuchardt88:master

Conversation

@cschuchardt88
Copy link
Copy Markdown

Change Log

  • Changed layout

Made it easier for you to develop. Ill update the code later. We are missing packages on nuget.org for neo-modules on new release. So I'll set-up this repo for you. Than I'll make the changes for you in neo-project/neo#3225 (comment)

If you don't want this just close this PR.

@Hecate2
Copy link
Copy Markdown
Owner

Hecate2 commented May 12, 2024

Please allow me to think twice on having a standalone Fairy directory, and making the dependency changes in Fairy.csproj. My original setup encourages users to clone this repo stupidly simply into neo-modules, and build and debug the whole source codes of Neo, including the RpcServer and Neo.Json.

@cschuchardt88
Copy link
Copy Markdown
Author

Why do you did that though? You can just use the nuget packages. It can very confusing for someone that doesn't use any code. that just expects you to open and compile/build.

@Hecate2
Copy link
Copy Markdown
Owner

Hecate2 commented May 12, 2024

Because nuget packages cannot be debugged, and there can be weird exceptions from anywhere. I do need to debug with source codes to capture errors.

@cschuchardt88
Copy link
Copy Markdown
Author

they have debug information in the nuget package. Just tell your IDE to enable nuget package source.
image

@Hecate2
Copy link
Copy Markdown
Owner

Hecate2 commented May 13, 2024

they have debug information in the nuget package. Just tell your IDE to enable nuget package source.

I think this is still making difficulties for running Fairy with any version and any commit of Neo source codes... I made Fairy for not only testing contracts, but evaluating contracts at any live version of blockchain.

@cschuchardt88
Copy link
Copy Markdown
Author

OK. Just thought your plugin was for neo-cli as a plugin.

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