Comments (2)
Hello. Thanks for opening an issue on Exercism 🙂
At Exercism we use our Community Forum, not GitHub issues, as the primary place for discussion. That allows maintainers and contributors from across Exercism's ecosystem to discuss your problems/ideas/suggestions without them having to subscribe to hundreds of repositories.
This issue will be automatically closed. Please use this link to copy your GitHub Issue into a new topic on the forum, where we look forward to chatting with you!
If you're interested in learning more about this auto-responder, please read this blog post.
from cli.
I make the change in my local
modified workspace/workspace.go
@@ -123,11 +123,11 @@ func (ws Workspace) Exercises() ([]Exercise, error) {
func (ws Workspace) ExerciseDir(s string) (string, error) {
if !strings.HasPrefix(s, ws.Dir) {
var err = fmt.Errorf("not in workspace")
- if runtime.GOOS == "darwin" {
- err = fmt.Errorf("%w: directory location may be case sensitive: workspace directory: %s, "+
- "submit path: %s", err, ws.Dir, s)
+ if runtime.GOOS != "darwin" {
+ return "", err
+ } else if !strings.HasPrefix(strings.ToLower(s), strings.ToLower(ws.Dir)) {
+ return "", err
}
- return "", err
}
path := s
And looks like it can handle both upper/lower case when I try to submit. (and the unit test failed ofc)
I am not sure if anywhere else use this method (I only find it referenced in cmd/submit.go
.
If you guys think this change is making sense, I can give a PR to make this change.
from cli.
Related Issues (20)
- `exercism configure --workspace=D:` panics HOT 2
- FR: Run analyzer locally before submitting
- Fetch next exercise HOT 5
- Token is invalid error. Tried multiple ones HOT 4
- Is the file size restriction still correct? HOT 1
- feature request: `exercism download` should warn the user if a newer version is available
- FR: add go install as install method
- Broken links in CLI repo's contributing guide HOT 2
- Discussion: Deliver VSCode devcontainer with Exercises HOT 2
- help text for submit
- `exercism open` should open the exercise in the browser when in exercise folder.
- Wrong URL for fetching user token. HOT 1
- Delivery Option: container.dev features. HOT 1
- [feature] support local configuration HOT 1
- Add support for retrieving API token from secure storage (read from env var?) HOT 7
- Release 3.1.0 does not include shell completions HOT 1
- Setup goreleaser to update homebrew HOT 6
- Automate releases HOT 7
- Build the cli ? HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from cli.