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

Include make commands #4

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Include make commands #4

wants to merge 3 commits into from

Conversation

MrYann
Copy link

@MrYann MrYann commented Apr 30, 2017

Hi.

I think it would be great to have the fortran extension providing some basic features like "make" out of the box (i.e. without installing third party all-purpose code runners).

Unfortunately, I have no real knowledge on how to program VSCODE, and don't know typescript. So I propose here a NON WORKING pull request, as a starting point for anyone willing to do the actual work, should you agree with the idea.

I looked into the code of "latex-workshop", which provides a "build" command and did some copy-paste :

  • package.json to declare the featured command (missing a default keybinding, like alt shift b, for build)
  • extension.ts to declare the functions at load time, and the actual functions.

I left in CAPITALS the parts that I don't know how to code.
If you are willing to implement this, it would be great. Of course, I'd be happy to help further if I can :)

@Gimly
Copy link
Owner

Gimly commented May 4, 2017

Hello,

Thanks for the work, very interesting. I will check it further and try to see what I can do to build something functional with it.

Cheers.

@MrYann
Copy link
Author

MrYann commented May 4, 2017 via email

@MrYann
Copy link
Author

MrYann commented Jul 17, 2017

Hi.

As you certainly know, VSCode now implements an auto indent feature where you only have to set indent and decent regex patterns. So, I propose you here a vary basic first try. It may be incomplete, or not optimally written, but works on simple examples.

Note that only the file modern.configuration.json has to be modified.

Also, the auto indent feature does not correct indentation within pasted blocs; which is a pity; so you have to write your own lines to test it.

@Gimly
Copy link
Owner

Gimly commented Sep 30, 2017

Hi there, I'm really sorry to not have made any comment to your great PR, not a great way to treat someone who took time to help...

I'm going to have finally a bit of time now to get your work some well deserved love and will try to add your contributions to the code. I don't think I'll be able to merge the PR directly though, as the first commits are non working.

@MrYann
Copy link
Author

MrYann commented Oct 1, 2017 via email

@Gimly
Copy link
Owner

Gimly commented Oct 2, 2017

About the idea to include a "Make" command, which compiler should be used for that?

@MrYann
Copy link
Author

MrYann commented Oct 2, 2017 via email

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