Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

languages/vala: init #432

Merged
merged 6 commits into from
Nov 3, 2024
Merged

languages/vala: init #432

merged 6 commits into from
Nov 3, 2024

Conversation

Soliprem
Copy link
Contributor

@Soliprem Soliprem commented Nov 1, 2024 •
edited
Loading

Add support for Vala language

Sanity Checking

  • I have updated the changelog as per my changes.
  • I have tested, and self-reviewed my code.
  • Style and consistency
    • I ran Alejandra to format my code (nix fmt).
    • My code conforms to the editorconfig configuration of the project.
    • My changes are consistent with the rest of the codebase.
  • If new changes are particularly complex:
    • My code includes comments in particularly complex areas
    • I have added a section in the manual.
    • (For breaking changes) I have included a migration guide.
  • Package(s) built:
    • .#nix (default package)
    • .#maximal
    • .#docs-html
  • Tested on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin

Add a 👍 reaction to pull requests you find important.

@Soliprem Soliprem requested a review from NotAShelf as a code owner November 1, 2024 23:05
@Soliprem Soliprem force-pushed the vala branch 2 times, most recently from b2cb200 to 96880fc Compare November 1, 2024 23:10
@Soliprem Soliprem changed the title Vala: init languages/vala: init Nov 2, 2024
diniamo
diniamo requested changes Nov 3, 2024
View reviewed changes
NotAShelf
NotAShelf requested changes Nov 3, 2024
View reviewed changes
Copy link
Owner

@NotAShelf NotAShelf left a comment

Choose a reason for hiding this comment

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

Alongside diniamo's changes.

@Soliprem Soliprem requested review from NotAShelf and diniamo November 3, 2024 12:29
diniamo
diniamo requested changes Nov 3, 2024
View reviewed changes
Copy link
Collaborator

@diniamo diniamo left a comment

Choose a reason for hiding this comment

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

You are also missing a release note entry

@Soliprem Soliprem requested a review from diniamo November 3, 2024 19:16
@diniamo diniamo merged commit 0e76316 into NotAShelf:v0.7 Nov 3, 2024
7 checks passed
@NotAShelf NotAShelf mentioned this pull request Nov 8, 2024
Merged
@github-actions github-actions bot deleted the vala branch December 15, 2024 04:11
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@diniamo diniamo diniamo approved these changes

@NotAShelf NotAShelf Awaiting requested review from NotAShelf NotAShelf is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Soliprem @diniamo @NotAShelf