Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@RichardDS90
Copy link
Contributor

@RichardDS90 RichardDS90 commented Dec 18, 2025

@RichardDS90 RichardDS90 marked this pull request as draft December 18, 2025 03:05
@kcat
Copy link
Contributor

kcat commented Dec 24, 2025

I can't help but be curious if these may have been intentional. Especially with many of them only affecting easy skill levels, it makes me wonder if they purposely made 100% impossible on easy skill levels (which doesn't seem too crazy of an idea to me?). If it was purposely made that way, I'm not sure it's on us to "fix" it, so it may be a good idea to get verification that it is unintentional.

@RichardDS90
Copy link
Contributor Author

I can't see any logical reason why that would be something that would be done. Especially considering I've seen a few map fixes that actually do allow monsters to teleport on all difficulties instead of just the harder ones.

@mc776
Copy link
Contributor

mc776 commented Dec 24, 2025

Deliberate aesthetic choice to make something look janky and incomplete for diegetic reasons I can get.

Deliberate gameplay choice to frustrate players for playing on easy, frankly the kind of person who intends that kind of thing's artistic vision is probably best summarily disregarded.

But also overwhelmingly unlikely compared to someone just never testing on easy and accidentally leaving something (un)flagged in an actor - a mistake I've seen and done not at all infrequently.

@the-phinet
Copy link
Contributor

Nice work :)

Can you make sure you rebase instead of merging when you pull in updates to this branch?

@RichardDS90
Copy link
Contributor Author

Nice work :)

Can you make sure you rebase instead of merging when you pull in updates to this branch?

Yeah sure thing, how do I do that? Not too familar with it.

@kcat
Copy link
Contributor

kcat commented Dec 24, 2025

I can't see any logical reason why that would be something that would be done.

To make it so easy skills are more limited, not unlike games that give different endings based on the skill level you play at. Maybe it's not too likely, but it doesn't seem that far fetched that the thought came to me, is all.

@the-phinet
Copy link
Contributor

Yeah sure thing, how do I do that? Not too familar with it.

It depends on how you are interfacing with git.

I do:

git rebase upstream/trunk
git push --force-with-lease

If you use github desktop, this looks like the instructions for it https://docs.github.com/en/desktop/working-with-your-remote-repository-on-github-or-github-enterprise/syncing-your-branch-in-github-desktop#rebasing-your-project-branch-onto-another-branch

Alternatively: When this pr is ready, I can fix the merge commits for you

@RichardDS90
Copy link
Contributor Author

Kama Sutra MAP15, if you're going to the secret map, 100% kills becomes impossible, and because of the nature of the mod, not sure if it's deliberate or not, worth having the monsters in the shootout area not count towards kills?

@the-phinet

This comment was marked as resolved.

@github-actions
Copy link

github-actions bot commented Jan 17, 2026

Thanks for opening this PR! We'll review it soon.

I'm a bot that checks for some common problems. I'll update this comment with anything I find.

Looks like it's your first time contributing!

  • Welcome the community and thank you for your contribution! Please take a moment to familiarize yourself with the resources available in our README.md file.

Nothing to report. Nice :)

@RichardDS90
Copy link
Contributor Author

@the-phinet Got a batch of map fixes all ready for merging. Will do another batch some other time. :)

@RichardDS90 RichardDS90 marked this pull request as ready for review January 17, 2026 12:04
@the-phinet
Copy link
Contributor

Nice! Can you add your signature to the bottom of the CONTRIBUTORS file?

@RichardDS90
Copy link
Contributor Author

Nice! Can you add your signature to the bottom of the CONTRIBUTORS file?

All done. :)

@the-phinet the-phinet enabled auto-merge (rebase) January 17, 2026 18:11
@the-phinet the-phinet merged commit 2bdd366 into UZDoom:trunk Jan 17, 2026
14 checks passed
@drfrag666
Copy link
Contributor

I see that there are many different commits with the same message. Besides fixes for the same wad used to be close to each other.

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.

5 participants