You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@devlake.apache.org by GitBox <gi...@apache.org> on 2022/06/02 09:42:47 UTC

[GitHub] [incubator-devlake-website] SnowMoon-Dev opened a new pull request, #62: [improve][dev setup doc] Replace the devlake module with the current project path

SnowMoon-Dev opened a new pull request, #62:
URL: https://github.com/apache/incubator-devlake-website/pull/62

   ## Purpose of the pull request
   
   ## Brief change log
   
   For easier local development
   
   ## Verify this pull request
   
   Manually verified the change by testing locally.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-devlake-website] klesh commented on a diff in pull request #62: [improve][dev setup doc] Replace the devlake module with the current project path

Posted by GitBox <gi...@apache.org>.
klesh commented on code in PR #62:
URL: https://github.com/apache/incubator-devlake-website/pull/62#discussion_r893240219


##########
docs/02-QuickStart/04-DeveloperSetup.md:
##########
@@ -31,26 +31,31 @@ description: >
     ```sh
 	go get
     ```
+   
+4. Replace the devlake module with the current project path

Review Comment:
   I see, did you try to restart your IDE? This looks like an IDE problem to me, not caused by DevLake or Golang itself.
   Will this operation modify any file like `go.mod` ?
   If it happens only for some IDEs, we should move it to `Trouble-shooting` section, and describe the consequence of this workaround to avoid unwanted files being committed accidentally



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-devlake-website] SnowMoon-Dev commented on a diff in pull request #62: [improve][dev setup doc] Replace the devlake module with the current project path

Posted by GitBox <gi...@apache.org>.
SnowMoon-Dev commented on code in PR #62:
URL: https://github.com/apache/incubator-devlake-website/pull/62#discussion_r894112797


##########
docs/02-QuickStart/04-DeveloperSetup.md:
##########
@@ -31,26 +31,31 @@ description: >
     ```sh
 	go get
     ```
+   
+4. Replace the devlake module with the current project path

Review Comment:
   I restarted to no avail, it would modify go.mod, but only for the first time, maybe this problem has something to do with my own IDE, but I'm not sure, I moved this problem to Trouble-shooting so that This method will help others if they encounter it



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-devlake-website] SnowMoon-Dev closed pull request #62: [improve][dev setup doc] Replace the devlake module with the current project path

Posted by GitBox <gi...@apache.org>.
SnowMoon-Dev closed pull request #62: [improve][dev setup doc] Replace the devlake module with the current project path
URL: https://github.com/apache/incubator-devlake-website/pull/62


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-devlake-website] SnowMoon-Dev commented on a diff in pull request #62: [improve][dev setup doc] Replace the devlake module with the current project path

Posted by GitBox <gi...@apache.org>.
SnowMoon-Dev commented on code in PR #62:
URL: https://github.com/apache/incubator-devlake-website/pull/62#discussion_r890904107


##########
docs/02-QuickStart/04-DeveloperSetup.md:
##########
@@ -31,26 +31,31 @@ description: >
     ```sh
 	go get
     ```
+   
+4. Replace the devlake module with the current project path

Review Comment:
   ![1654589341666](https://user-images.githubusercontent.com/42627702/172329810-bd6ff631-df9c-43d4-acc6-fa687e5a61ad.jpg)
   
   go mod edit -replace github.com/apache/incubator-devlake=./
   
   <img width="975" alt="image" src="https://user-images.githubusercontent.com/42627702/172330014-207995c4-48d5-4b53-9382-64c539d15b6d.png">
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-devlake-website] SnowMoon-Dev commented on a diff in pull request #62: [improve][dev setup doc] Replace the devlake module with the current project path

Posted by GitBox <gi...@apache.org>.
SnowMoon-Dev commented on code in PR #62:
URL: https://github.com/apache/incubator-devlake-website/pull/62#discussion_r889964032


##########
docs/02-QuickStart/04-DeveloperSetup.md:
##########
@@ -31,26 +31,31 @@ description: >
     ```sh
 	go get
     ```
+   
+4. Replace the devlake module with the current project path

Review Comment:
   This is primarily for first-time build users, according to the official website after installation, can work normally, but IDE such as my Goland will show the import dependency package error



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-devlake-website] klesh commented on a diff in pull request #62: [improve][dev setup doc] Replace the devlake module with the current project path

Posted by GitBox <gi...@apache.org>.
klesh commented on code in PR #62:
URL: https://github.com/apache/incubator-devlake-website/pull/62#discussion_r893240219


##########
docs/02-QuickStart/04-DeveloperSetup.md:
##########
@@ -31,26 +31,31 @@ description: >
     ```sh
 	go get
     ```
+   
+4. Replace the devlake module with the current project path

Review Comment:
   I see, did you try to restart your IDE? This looks like a workaround for IDE to me, not a legitimate problem of DevLake or Golang itself.
   Will this operation modify any file like `go.mod` ?
   If it happens only for some IDEs, we should move it to `Trouble-shooting` part, and describe the consequence of this workaround.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-devlake-website] SnowMoon-Dev commented on a diff in pull request #62: [improve][dev setup doc] Replace the devlake module with the current project path

Posted by GitBox <gi...@apache.org>.
SnowMoon-Dev commented on code in PR #62:
URL: https://github.com/apache/incubator-devlake-website/pull/62#discussion_r889964032


##########
docs/02-QuickStart/04-DeveloperSetup.md:
##########
@@ -31,26 +31,31 @@ description: >
     ```sh
 	go get
     ```
+   
+4. Replace the devlake module with the current project path

Review Comment:
   Mainly for a build of the user, according to the official website after installation, can work normally, but IDE such as my Goland will show the import dependency package error



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-devlake-website] klesh commented on a diff in pull request #62: [improve][dev setup doc] Replace the devlake module with the current project path

Posted by GitBox <gi...@apache.org>.
klesh commented on code in PR #62:
URL: https://github.com/apache/incubator-devlake-website/pull/62#discussion_r889949227


##########
docs/02-QuickStart/04-DeveloperSetup.md:
##########
@@ -31,26 +31,31 @@ description: >
     ```sh
 	go get
     ```
+   
+4. Replace the devlake module with the current project path

Review Comment:
   Do we need this? I have setup devlake multiple times, it worked fine without this operation.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org