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

File-based config .approvalsConfig not behaving as expected #71

Closed
cmstead opened this issue Mar 7, 2017 · 5 comments
Closed

File-based config .approvalsConfig not behaving as expected #71

cmstead opened this issue Mar 7, 2017 · 5 comments

Comments

@cmstead
Copy link

cmstead commented Mar 7, 2017

I tried migrating my configuration to a file-based config as suggested in the docs, but it looks like my custom reporter is not recognized. I'm not clear if my format is simply wrong and the config is not Javascript or if custom reporters are not being recognized. Perhaps the config file isn't being read? I'm just really unclear on where the actual problem lives. Let me know what info you need to help identify the cause and I'll gather it for you.

@staxmanade
Copy link
Member

Go ahead and share your file and describe where this file is located. Also what version of approvals are you using?

@cmstead
Copy link
Author

cmstead commented Mar 8, 2017

I'll commit an example to the same branch as the other project and post a link here in a moment.

@cmstead cmstead closed this as completed Mar 8, 2017
@cmstead
Copy link
Author

cmstead commented Mar 8, 2017

Correction, the approvals config example is already in the branch. Look in the root folder and you should see the .approvalsConfig file:

https://github.com/cmstead/js-refactor/tree/approvalsIssue

Also, thanks for digging into this!

(I didn't mean to close this.)

@cmstead cmstead reopened this Mar 8, 2017
@staxmanade
Copy link
Member

ahh yea - sorry... While it's totally obvious that's how it "should" work... sadly, that's not how it does.

Issue #56 is probably what you're looking for and we haven't built that out yet...

If you want, you can copy the .approvalsConfig file into your "home" directory so on windows C:/Users/yourName/.approvalsConfig - this only fixes it for your computer.

If you feel interested in taking a stab at implementing #56 I'll be here to support it. Not sure when I'll get around to it at the moment...

@cmstead
Copy link
Author

cmstead commented Mar 10, 2017

Sorry for the delay. I'll have a look at the issue and see what, if anything, I might be able to do. : )

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

No branches or pull requests

2 participants