Skip to content

Update repository configuration and resolve master merge conflicts#272

Open
th3w1zard1 wants to merge 281 commits into
masterfrom
bleeding-edge
Open

Update repository configuration and resolve master merge conflicts#272
th3w1zard1 wants to merge 281 commits into
masterfrom
bleeding-edge

Conversation

@th3w1zard1
Copy link
Copy Markdown
Collaborator

@th3w1zard1 th3w1zard1 commented May 24, 2026

Description

This PR updates repository configuration, contributor conventions, and development environment setup files, and also resolves the branch's merge conflicts against master by merging the latest base branch into bleeding-edge.

These changes improve repository structure, contributor onboarding, and automation guidance while keeping the PR branch mergeable.

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📚 Documentation update
  • 🔧 Refactoring (no functional changes)
  • ⚡ Performance improvement
  • ✅ Test addition or update
  • 🔨 Build system or CI changes
  • 🎨 Code style changes (formatting, renaming, etc.)

Package(s) Affected

  • PyKotor (core library)
  • PyKotorGL
  • PyKotorFont
  • HolocronToolset
  • HoloPatcher
  • BatchPatcher
  • KotorDiff
  • GuiConverter
  • Documentation
  • CI/CD
  • Other: repository configuration and contributor tooling

Changes Made

  • Added a comprehensive .cursorrules file covering contributor and AI-agent workflow expectations.
  • Updated .env to standardize PYTHONPATH, K1_PATH, and K2_PATH.
  • Added .cursor/worktrees.json for OS-specific and generic worktree setup flows.
  • Added .github/CODEOWNERS to define ownership for repository areas and configuration.
  • Updated .cursorignore to better match repository structure.
  • Removed the obsolete .archivist/config.json.
  • Merged the latest master branch into bleeding-edge to resolve PR merge conflicts and restore branch mergeability.

Testing

  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have tested this manually
  • I have tested on multiple Python versions (3.8+)
  • I have tested on multiple operating systems (if applicable)

Test Results

- git diff --check: passed
- Verified the merge completed cleanly with no remaining unmerged paths
- Targeted pytest execution could not be run in this sandbox because pytest was not installed

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

Related Issues

Screenshots (if applicable)

N/A

Additional Notes

The merge conflict resolution was completed by merging origin/master into bleeding-edge and keeping the PR branch's intended conflicting changes where required.

th3w1zard1 and others added 30 commits October 10, 2024 18:14
refactor: Integrate Panda3D and fix OpenGL-related errors in scene.py
```
- Included additional extensions for enhanced Qt development support.
- Added extensions for AI coding assistance and code syncing features.
- Improved the overall functionality and user experience in the development environment.
…structure

- Updated the 'swizzle' and 'deswizzle' methods to use clearer parameter formatting.
- Enhanced documentation for both methods, providing detailed parameter descriptions.
- Improved the organization of the 'swizzle_offset' and 'deswizzle_offset' functions for better readability.
- Added type hints for variables to enhance code readability and maintainability.
- Removed unnecessary comments and dead code to streamline the implementation.
- Updated the 'TPCBinaryWriter' class to include clearer documentation on the file structure.
- Improved the handling of dimensions and mipmap generation logic in the 'TPC' class.
- Removed a large block of unused JavaScript code that was left in the repository.
- Removed several unused extensions from the list of recommended extensions.
- Updated Python formatting and linting configurations, including line length adjustments.
- Added new settings for 'ruff', 'black', and 'isort' to enhance code style consistency.
- Modified 'pyright' settings for improved type checking and analysis.
- Adjusted testing configurations to include new patterns for exclusions.
- This change cleans up the list of extensions by removing an unnecessary entry.
- The removal may help in reducing clutter and improving clarity in the configuration.
th3w1zard1 and others added 11 commits December 2, 2025 20:40
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Resolved all merge conflicts by preferring bleeding-edge over master:
- Kept deletions from bleeding-edge (modify/delete conflicts)
- Used bleeding-edge version for all content conflicts (UU)
- Used bleeding-edge version for add/add conflicts (AA)
- Resolved file location conflicts in favor of bleeding-edge structure
Incorporated relevant commits from NickHugi/PyKotor:master:
- Performance optimizations for Module Designer loading
- Module designer performance/controls improvements
- Error handling for 2da row limits
- Updated .gitignore for utility updater script

Resolved conflicts by:
- Keeping .cursorrules (deleted in upstream)
- Keeping deletions of scene.py and dlg.py from bleeding-edge
- Merging performance improvements from upstream
- Adding update.py to .gitignore
Remove leftover conflict markers that blocked Forward Commits to
Bleeding-Edge from checking out the branch after master merges.
@th3w1zard1
Copy link
Copy Markdown
Collaborator Author

@copilot resolve the merge conflicts in this pull request

Comment thread Libraries/PyKotor/src/pykotor/merge/module.py Fixed
Comment thread Libraries/PyKotor/src/pykotor/tools/__init__.py Fixed
Comment thread Libraries/PyKotor/src/pykotor/tools/__init__.py Fixed
Comment thread Libraries/PyKotor/src/pykotor/tools/__init__.py Fixed
Comment thread Libraries/PyKotor/src/pykotor/tools/__init__.py Fixed
Comment thread Libraries/PyKotor/src/pykotor/tools/__init__.py Fixed
Comment thread Libraries/PyKotor/src/pykotor/tools/__init__.py Fixed
Comment thread Libraries/PyKotor/src/pykotor/tools/__init__.py Fixed
Comment thread Libraries/PyKotor/src/pykotor/tools/__init__.py Fixed
Comment thread Libraries/PyKotor/src/pykotor/tools/__init__.py Fixed
* docs(plans): add bleeding-edge gitmodules fix plan

Record plan 020 for resolving merge conflict markers on bleeding-edge
and mark plan 019 completed after PR #268 merge.

* docs(plans): record post-merge verification for plan 019

Add verification table for PR #268 merge and pending PyPI regression run.

* docs(plans): finalize pypi regression post-268 verification

* docs(plans): record bleeding-edge gitmodules closeout verification

Document R1-R4 evidence: fix f15e476 on origin/bleeding-edge, parse
validation, and forward-commits re-run status.
Agent-Logs-Url: https://github.com/OpenKotOR/PyKotor/sessions/d80c584a-c16d-4b6c-8e9d-2f6a9dd199d8

Co-authored-by: th3w1zard1 <2219836+th3w1zard1@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI commented May 24, 2026

@copilot resolve the merge conflicts in this pull request

Fixed in e1b1959.

Comment thread Tools/HoloGenerator/web/src/App.tsx Fixed
Comment thread Tools/HoloGenerator/web/src/__tests__/useHistory.test.ts Fixed
Copy link
Copy Markdown
Contributor

@github-advanced-security github-advanced-security AI left a comment

Choose a reason for hiding this comment

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

CodeQL found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

Land Last CI check section and plan 056 verification table; add
AGENTS defer rule when ci-status-only matches checkpoint.
@th3w1zard1 th3w1zard1 marked this pull request as ready for review May 28, 2026 00:37
@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@th3w1zard1
Copy link
Copy Markdown
Collaborator Author

@copilot resolve the merge conflicts in this pull request

Copy link
Copy Markdown
Contributor

Copilot AI commented May 28, 2026

@copilot resolve the merge conflicts in this pull request

Fixed in b2c7c47.

Copilot AI changed the title Update README with test results and status badges Update repository configuration and resolve master merge conflicts May 28, 2026
self.ignoring_updates = False


class DLGLinkSync(DLGLink):
return True


class CrashDialog(PythonQFileDialog):
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.

4 participants