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

docs: Add example workspace #44

Merged
merged 7 commits into from
May 21, 2020
Merged

docs: Add example workspace #44

merged 7 commits into from
May 21, 2020

Conversation

michalbali256
Copy link
Contributor

No description provided.

* STATEMENTS
* Source code is divided into statements
* Each statement constists of:
* - label/name (optional, yellow)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

label is not always optional. I would get rid of optional and mandatory field info at all.

example_workspace/tutorial.hlasm Outdated Show resolved Hide resolved
example_workspace/tutorial.hlasm Outdated Show resolved Hide resolved
example_workspace/tutorial.hlasm Outdated Show resolved Hide resolved
* processor group P1 is assigned to this file (tutorial)
* P1 has 2 library folders - libs and libs2 - their files may be used

* MAC2 is defined in libs/mac2 and is invoked by inserting its name
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

inserting? writing

@@ -0,0 +1,35 @@

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why empty lines?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also please make all symbols upper case

@fox0r fox0r added this to the 0.12.0 milestone May 19, 2020
@michalbali256 michalbali256 requested a review from asmelko May 20, 2020 16:52
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Contributor

@asmelko asmelko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, thats epic

@michalbali256 michalbali256 changed the title chore: Add example workspace docs: Add example workspace May 21, 2020
@michalbali256 michalbali256 merged commit 774bb1d into development May 21, 2020
@michalbali256 michalbali256 deleted the example_workspace branch June 3, 2020 19:19
@github-actions
Copy link

🎉 This PR is included in version 0.12.0-beta.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions
Copy link

🎉 This PR is included in version 0.12.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

SWETAS04 pushed a commit to SWETAS04/che-che4z-lsp-for-hlasm that referenced this pull request Feb 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants