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

Day1 Advpl problem FizzBuzz #347 #356

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

Conversation

diogenesdauster
Copy link

@diogenesdauster diogenesdauster commented Oct 9, 2020

Please make sure you have done the following:

  • Created a folder with the name of language you code inside directory for the day you are making the contribution to
  • Added the source code file inside /Day1/Outsystems/
  • Tested your code
  • Added credits (author and date) at the top of the code
  • Updated README.md for /Day1/
  • Added yourself as a contributor using all-contributors-cli

@diogenesdauster diogenesdauster changed the title Day1 Ousystems problem FizzBuzz [#347](https://github.com/CodeToExpress/dailycodebase/issues/347) Day1 Ousystems problem FizzBuzz #347 Oct 9, 2020
Copy link
Contributor

@aaditkamat aaditkamat left a comment

Choose a reason for hiding this comment

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

Hi @diogenesdauster, the purpose of this repository is for contributors to solve the problems listed by implementing algorithms using a programming language. From what I see, OutSystems is not a programming language, but a software that allows you to draft low code solutions by reasoning about the problems at a higher level. This is a great way of thinking about problems as you come up with the algorithms, but ultimately you should express the solution in code. Thank you very much for presenting a different solution and hope you can see why I have to request changes for this PR.

@diogenesdauster
Copy link
Author

@aaditkamat as you said I've made some changes for the other language that I work on Advpl, because Outsystems does not have code it's visual implementation. Could you look if it ok now ?

@diogenesdauster diogenesdauster changed the title Day1 Ousystems problem FizzBuzz #347 Day1 Advpl problem FizzBuzz #347 Oct 18, 2020
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