Skip to content

Fix some tutorial bits - #2575

Merged
Perksey merged 4 commits into
dotnet:mainfrom
aquagoose:fix-tutorial-stuff
Aug 2, 2026
Merged

Fix some tutorial bits#2575
Perksey merged 4 commits into
dotnet:mainfrom
aquagoose:fix-tutorial-stuff

Conversation

@aquagoose

@aquagooseaquagoose commented May 1, 2026

Copy link
Copy Markdown
Collaborator

Summary of the PR

  • Fixes the Texture tutorial so it uses glTexParameter instead of glTextureParameter
  • Replaces uses of StbImageSharp with ImageSharp
    • This was done because all the other tutorials use ImageSharp, and at the time when I wrote the texture tutorial I decided to replace ImageSharp with StbImageSharp for some reason, but only in the texture tutorial and the existing abstraction tutorial which makes no sense. Also there was a lot of pushback after it was merged after the team noticed it had been replaced, so I'm righting the wrong here.

Related issues, Discord discussions, or proposals

Links go here.

Further Comments

@ExaniteExanite left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me. We can merge once we get a review from Beyley.

I ran the examples locally on Linux and they work fine.
Behavior also matches the main branch.

Comment threadexamples/CSharp/OpenGL Tutorials/Tutorial 1.4 - Abstractions/Texture.cs Outdated

@PerkseyPerksey left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm happy to bless this on behalf of Beyley

@Exanite

Copy link
Copy Markdown
Member

@Perksey Feel free to merge. I don't want to merge this myself since I added an extra commit above.

@Perksey
Perksey merged commit 57e0f86 into dotnet:mainAug 2, 2026
3 checks passed
@github-project-automationgithub-project-automationBot moved this from In Progress to Done in Silk.NET Development TrackerAug 2, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants

@aquagoose@Exanite@Perksey