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

added Kotlin data types and control structures #732

Merged
merged 1 commit into from
Jul 29, 2024

Conversation

naeem4633
Copy link
Contributor

Changes done to add Control Structures and Data Types for Kotlin.

What GitHub issue does this PR apply to?

Continues the work from #728.

What changed and why?

  • Added a control_structures.json file inside the kotlin/1.5 directory containing the necessary content to define various control structures in Kotlin.
  • Added a data_types.json file inside the kotlin/1.5 directory containing the necessary content to define various data types in Kotlin.

(If editing website code) Please add screenshots

Not applicable.

Checklist

  • I claimed any associated issue(s) and they are not someone else's
  • I have looked at documentation to ensure I made any revisions correctly
  • I tested my changes locally to ensure they work
  • For language files, I have validated the edited files are valid JSON and data shows up correctly
  • For website code edits, I have added or edited any appropriate unit tests for my changes

Any additional comments or things to be aware of while reviewing?

None.

Copy link
Member

@geekygirlsarah geekygirlsarah left a comment

Choose a reason for hiding this comment

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

I think these look good. I'll merge them in too!

Thanks for adding more Kotlin categories. I'm going to add Kotlin issues for things tonight.

@geekygirlsarah geekygirlsarah merged commit e553c7f into codethesaurus:main Jul 29, 2024
5 checks passed
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