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

PureBasic #2362

Closed
HeX0R101 opened this issue May 4, 2020 · 9 comments
Closed

PureBasic #2362

HeX0R101 opened this issue May 4, 2020 · 9 comments

Comments

@HeX0R101
Copy link
Contributor

HeX0R101 commented May 4, 2020

Language
A language based on Basic (BlitzBasic to be precise) with inline assembler also and direct API calls. Can compile on Windows, Linux and MacOS

Additional resources
web site: https://www.purebasic.com
Documentation: https://www.purebasic.com/documentation/

I already started to create it on my own, but I'm a real moron, when talking about regex, and I also borroght some parts from a resource I found.
So it is far away from being perfect, but on the test side it works quite good.
npm run test:languages -- --language=purebasic --pretty also didn't show any problems, but npm test errors our, and I have no idea why.
I've attached all I have done so far.

Thanks!

BR
Tom
prism_purebasic.zip

@RunDevelopment
Copy link
Member

npm run test:languages -- --language=purebasic --pretty also didn't show any problems, but npm test errors our, and I have no idea why.

npm run test:languages basically just ensure that your language works and that it's correct. npm test also includes some really strict tests for regexes, patterns, and names. I added these in because they are all things that can be potential errors and I will them point out in review anyway.
Could you please tell me what error you got? (I want to add a more descriptive error message.)

@HeX0R101
Copy link
Contributor Author

HeX0R101 commented May 4, 2020

Please see attached
npm_test_errorscreen
2020-05-04T10_15_45_316Z-debug.log

@RunDevelopment
Copy link
Member

Is this error message clearer?

image

(I'm sorry that I don't just tell you what to do here but I really want that you and others are able to understand why this error is there and how to fix it.)

@HeX0R101
Copy link
Contributor Author

HeX0R101 commented May 4, 2020

No need to apologize, I'm fine with showing me the door and letting me walk through alone.
Yes, that description should help indeed!
Let me see, if I can make it shut-up now.
Any chance to make the red color of the error messages a little lighter? It's quite hard to read...

@RunDevelopment
Copy link
Member

The was just a little screenshot. #2364 is now in master, so you can get the changes and read the errors in your favorite color 😄.

If you have any other questions or can't make some errors disappear, feel free to ask. My little experiment is now over.

@HeX0R101
Copy link
Contributor Author

HeX0R101 commented May 5, 2020

I'm not very familar with git... but I think I made the language description work.
I've cloned the master, and integrated the changes to my local copy.
Can I just "push" the changes to the master, or will I somehow destroy your master then?

@RunDevelopment
Copy link
Member

I will take "integrated" as committed. You can push the master branch to your fork, that will work. (Not ideal but simple)

@RunDevelopment
Copy link
Member

Also, you generally can't destroy (as in lose data) anything with git. (Technically, you can but you have to try really hard for that.)

@RunDevelopment
Copy link
Member

#2369

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants