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

Now runs in Windows #593

Merged
merged 1 commit into from
Jul 15, 2023
Merged

Now runs in Windows #593

merged 1 commit into from
Jul 15, 2023

Conversation

A-H4NU
Copy link
Contributor

@A-H4NU A-H4NU commented Jul 12, 2023

Prelude

Thank you for helping out vim-doge!

By contributing to vim-doge you agree to the following statements:

Why this PR?

This was also an issue for me.

Adding /bin/vim-doge-helper.exe to let l:executables in the function doge#run_parser() resolved the issue.

@auto-assign auto-assign bot requested a review from kkoomen July 12, 2023 17:24
@A-H4NU A-H4NU changed the title make it at least run Now runs in Windows Jul 12, 2023
@kkoomen kkoomen merged commit 07b85d6 into kkoomen:master Jul 15, 2023
13 checks passed
@kkoomen
Copy link
Owner

kkoomen commented Jul 15, 2023

Hi!

Thanks, I totally forgot that indeed, appreciated!

@isfandyarIOT
Copy link

can you guide me how you solved the issue ?

@A-H4NU
Copy link
Contributor Author

A-H4NU commented Jul 19, 2023

@isfandyarIOT Edit the file autoload/doge.vim like this. It resolved my issue. :)

@isfandyarIOT
Copy link

@isfandyarIOT Edit the file autoload/doge.vim like this. It resolved my issue. :)

thank you it works now test both python and c file

@kkoomen
Copy link
Owner

kkoomen commented Jul 24, 2023

I will do a release after 2 weeks, as I'm currently on a holiday trip. After 2 weeks I'll let you know here about 4.0.1 that contains this updated fix.

@kkoomen kkoomen mentioned this pull request Jul 24, 2023
@kkoomen
Copy link
Owner

kkoomen commented Jul 26, 2023

This has been merged in a new version v4.1.0, enjoy!

Thanks for the contribution, it's been appreciated.

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