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

DIAP - Addressing issue 1-3 - adding new separator - dealing with NLS charac… #23

Merged
merged 2 commits into from
Nov 14, 2022

Conversation

stephanweigandt
Copy link
Contributor

…ter correctly by using k-functions - separator detection fix

Please include answers to these questions as part of your pull request

In the GitHub webUI, use the Write tab to modify the Markdown text that is part of the pull request. For each question simply place an X inside the square brackets, [X], that represents your answer. Using the Preview tab you can see the formatted/rendered version of the message.

  • Q1: Confirm that you have the right to submit the code that is being contributed. Please consider the origin of your code and confirm you have the appropriate rights to make the submission subject to the Apache 2.0 license that applies to everything in this repository of custom steps. If so, follow the instructions for the Contributor Agreement (which is based on the industry-standard Developer Certificate of Origin (DCO)).
    • Yes, I have the right to submit the contributed code on behalf of myself, my company, or any other owner of the code. I have also attached my signed copy of the DCO to this message.
    • No
  • Q2: Confirm that your contribution does not include any personally identifiable information (PII), for example, in any examples used in your README file.
    • My contribution does NOT include PII data
    • My contribution includes PII data
  • Q3: Confirm your contribution does not include any encryption or other export-controlled technology.
    • My contribution does NOT contain encryption or other export-controlled technology
    • My contribution includes encryption or other export-controlled technology
      contributing_agreement.txt

…ter correctly by using k-functions - separator detection fix
@snlwih
Copy link
Collaborator

snlwih commented Nov 11, 2022

11NOV22: Review started.

…ter correctly by using k-functions - separator detection fix
@stephanweigandt
Copy link
Contributor Author

the indenting is corrected now, please check again. Thank you!

@snlwih
Copy link
Collaborator

snlwih commented Nov 14, 2022

Lines containing SAS code are now nicely indented.
Lines containing comments are mostly nicely indented and seem to follow a consistent pattern, making them easy to read. There are a few comment blocks that do not adhere to that pattern and seem to be fully left-aligned. Not sure if that is an oversight. Any, will merge this pull request now so we can make the enhanced functionality available

@snlwih
Copy link
Collaborator

snlwih commented Nov 14, 2022

SAS Internal Due Diligence Review - COMPLETED on 14NOV2022

@snlwih snlwih merged commit 3b747f7 into sassoftware:main Nov 14, 2022
@snlwih snlwih changed the title Addressing issue 1-3 - adding new separator - dealing with NLS charac… DIAP - Addressing issue 1-3 - adding new separator - dealing with NLS charac… Nov 30, 2022
@snlwih snlwih added the update custom step This pull request updates an existing custom step label Dec 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
update custom step This pull request updates an existing custom step
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants