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

use source location for the LOG #151

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

mgiroday
Copy link
Contributor

  • fix: handle escaped characters when reading XML files
  • feat: parse ribbon description files for RC files
  • ci: Remove the reference to base_ref in docs job
  • .codespellrc - add codespell configuration
  • fix: deleteLine method name in logging
  • chore: cleanup gitreview config file
  • docs: clarify the difference between Project::get and Project::open
  • fix: cleanup the Ribbon API and documentation
  • ci: Install plantuml plugin before building docs
  • feat: Create a JsonDocument
  • fix: Fix message map extraction when namespaces are used
  • fix: put everything from utils/strings.h into the namespace Utils
  • chore: Reduce workspace extensions for VSCode
  • feat: add option to return the settings from the CLI
  • docs: Cleanup documentation
  • feat: add API to create a ui file in QtUiDocument
  • feat: Add utility class to help write Qt Ui files
  • refactor: move back the frame conversion to code
  • refactor: remove the rc2ui.js file, conversion is now static
  • refactor: increment duplicate ids in RcFile
  • fix: rename strings.h into string_helper.h
  • feat!: refactor the ScriptDialog API for progressbar
  • chore: rename ConversionProgressDialog into ScriptProgressDialog
  • feat!: Remove the uiFilePath property from ScriptDialog
  • test: add tests for the examples
  • feat: add support for QmlJs via tree-sitter-qmljs
  • refactor(cmake): Move version into version.txt
  • fix(tests): update mfc-utils for tests to pass
  • chore: Cleanup unused includes
  • chore: Fix std::move usage with const variables
  • chore: Remove unescessary ;
  • chore: Remove some copy, use const & when needed
  • chore: Multiple small cleanups
  • chore: Const-ify some methods/variables
  • fix: use /O3 with MSVC rather than -O3
  • fix: adapt optimization compilation to microsoft compiler
  • chore: Cleanup CMake targets
  • fix: use correct variable for toolButton
  • docs(example): example script to get/set data
  • feat: introduce Qml to parser
  • fix: we build against qt6 only now (fix: we build against qt6 only now #78)
  • fix: Remove version from import
  • chore: Add missing license to examples
  • feat: Add ribbon to the RcViewer
  • feat: add a ribbon model for rcviewer"
  • fix: default ui windowTitle is "Form" => it can be translated => remove it (fix: default ui windowTitle is "Form" => it can be translated => remove it #81)
  • ci: Add release-please workflow
  • fix(build): Fix debug build on Windows
  • fix: modification of the setTranslation function for the case where the node is untranslated
  • fix: const'ify value in knutstyle
  • fix: const'ify pointer
  • fix: in qt6 QVector is an alias to QList
  • refactor: reduce LSP dependencies in Core
  • docs: review the Utils API
  • docs: remove old LSP API
  • fix(docs): Fix an issue with array<> in doc
  • docs: Review the item's API
  • docs: Review the Document and Project API
  • refactor: Add logging for UserDialog return values
  • chore(ci): Use GitHub upload-pages-artifact and deploy-pages actions
  • feat: allow to search string in qttsview
  • refactor!: Rename Script QML module to Knut
  • chore(ci): Remove pre_commit job from GH Actions
  • fix: Use Qt_QPA_PLATFORM=offscreen for testing
  • fix: add stdset="0" we don't want to generate property
  • chore(cmake): Re-enable docs target by default
  • refactor!: Change the test API for scripts
  • chore: update mfc-utils and photonwidgets sub-modules
  • fix: allow to replace by empty string (=> remove text)
  • fix: save settings file in different directory in testing mode
  • fix: Fix saving stringlist in Settings via script
  • feat: Select Smaller/Larger/Next/Previous Syntax Nodes (feat: Select Smaller/Larger/Next/Previous Syntax Nodes #105)
  • chore: Bump version of photonwidgets
  • docs: Review QtTsDocument API
  • fix: Add missing properties to ScriptRunner::isProperty
  • docs: Review RcDocument API
  • refactor!: Remove TextLocation class
  • refactor!: Remove TextRange class
  • feat: Add a code view with button for TreeSitter
  • refactor: Improve document toolbar
  • chore: Add missing licenses for the icons used inside the application
  • fix: Fix crash in TreeSitter inspector with non-code document
  • fix: Delete the TreeSitter inspector when closed
  • feat: Skip macros in C++ during parsing (feat: Skip macros in C++ during parsing #116)
  • feat: added line property for a message node (feat: added line property for a message node #122)
  • feat: add a method to change the context of a message (feat: add a method to change the context of a message #121)
  • chore(docs): Fix & Improve contribution docs
  • chore: update mfc-utils sub-module (chore: update mfc-utils sub-module #124)
  • feat: Add symbol listing for Qml (feat: Add symbol listing for Qml #79)
  • fix: save settings on exit (fix: save settings on exit #125)
  • chore: update mfc-utils sub-module
  • feat: improve nextStep API in ScriptDialog (feat: improve nextStep API in ScriptDialog #129)
  • feat: Add TypedSymbol
  • fix: Capturing of return types in FunctionSymbol
  • build: Fix include issue with TreeSitter and unicode
  • fix: do not run tests if the path to the directory doesn't exist
  • fix: improve script performance by changing QTreeView settings
  • fix: Make Symbols & query... calls consistent
  • fix: simplify the API documentation (fix: simplify the API documentation #132)
  • feat: Scheme syntax highlighting for TreeSitter
  • fix!: Remove treesitter transformations
  • feat: add a way to initialize script dialog from command line (feat: add a way to initialize script dialog from command line #136)
  • fix: change syntax of comments to remove warning (fix: change syntax of comments to remove warning #139)
  • chore: update mfc-utils sub-module (chore: update mfc-utils sub-module #140)
  • feat: Handle QTextEdit in the ScriptDialogItem (feat: Handle QTextEdit in the ScriptDialogItem #141)
  • fix: flush spdlog messages also when logging to stderr (fix: flush spdlog messages also when logging to stderr #142)
  • fix: exit knut if there is an error in qml test (fix: exit knut if there is an error in qml test #147)
  • fix: remove unused forward declaration
  • docs: Note Windows build issues, provide steps to resolve (docs: Note Windows build issues, provide steps to resolve #144)
  • fix: allows describing type documentation with nested brackets and commas
  • feat: Add reuse and fix lint issues
  • feat: Saving & loading TS queries in inspector
  • feat: Custom treesitter grammar
  • refactor: Use std::source_location for the LOG

@mgiroday mgiroday force-pushed the use-source_location-for-the-LOG branch 2 times, most recently from 0637adc to 9372b2b Compare August 16, 2024 08:58
src/core/knutcore.cpp Outdated Show resolved Hide resolved
src/core/qttsdocument.cpp Outdated Show resolved Hide resolved
src/core/qtuidocument.cpp Outdated Show resolved Hide resolved
src/core/scriptmanager.cpp Outdated Show resolved Hide resolved
src/core/scriptmanager.cpp Outdated Show resolved Hide resolved
src/core/scriptmanager.cpp Outdated Show resolved Hide resolved
src/core/scriptrunner.cpp Outdated Show resolved Hide resolved
src/core/scriptrunner.cpp Outdated Show resolved Hide resolved
src/core/scriptrunner.cpp Outdated Show resolved Hide resolved
src/core/scriptrunner.cpp Outdated Show resolved Hide resolved
src/rccore/lexer.cpp Outdated Show resolved Hide resolved
src/rccore/rc_convert.cpp Outdated Show resolved Hide resolved
src/rccore/rc_convert.cpp Outdated Show resolved Hide resolved
src/rccore/rc_convert.cpp Outdated Show resolved Hide resolved
src/rccore/rc_convert.cpp Outdated Show resolved Hide resolved
src/rccore/rc_convert.cpp Outdated Show resolved Hide resolved
src/rccore/rc_convert.cpp Outdated Show resolved Hide resolved
src/rccore/rc_convert.cpp Outdated Show resolved Hide resolved
src/rccore/rc_parse.cpp Outdated Show resolved Hide resolved
src/rccore/rc_parse.cpp Outdated Show resolved Hide resolved
src/rccore/rc_parse.cpp Outdated Show resolved Hide resolved
src/rccore/rc_parse.cpp Outdated Show resolved Hide resolved
src/rccore/rc_parse.cpp Outdated Show resolved Hide resolved
src/rccore/rc_parse.cpp Outdated Show resolved Hide resolved
src/rccore/rc_parse.cpp Outdated Show resolved Hide resolved
src/rccore/rc_parse.cpp Outdated Show resolved Hide resolved
src/rccore/rc_parse.cpp Outdated Show resolved Hide resolved
src/rccore/rc_parse.cpp Outdated Show resolved Hide resolved
src/rccore/rc_parse.cpp Outdated Show resolved Hide resolved
src/rccore/rc_parse.cpp Outdated Show resolved Hide resolved
src/rccore/rc_parse.cpp Outdated Show resolved Hide resolved
src/rccore/rc_parse.cpp Outdated Show resolved Hide resolved
src/rccore/rc_parse.cpp Outdated Show resolved Hide resolved
src/rccore/ribbon.cpp Outdated Show resolved Hide resolved
src/treesitter/predicates.cpp Outdated Show resolved Hide resolved
src/utils/log.h Outdated Show resolved Hide resolved
src/utils/log.h Outdated Show resolved Hide resolved
src/utils/log.h Outdated Show resolved Hide resolved
src/utils/log.h Show resolved Hide resolved
src/treesitter/predicates.cpp Outdated Show resolved Hide resolved
@mgiroday mgiroday force-pushed the use-source_location-for-the-LOG branch 3 times, most recently from b3b3992 to fc65df2 Compare August 22, 2024 04:38
Copy link
Member

@narnaud narnaud left a comment

Choose a reason for hiding this comment

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

Sorry for the delay reviewing this patch.
Some small changes, but most of the work is done. Note that you need to rebase your work on the latest main.

src/core/codedocument.cpp Outdated Show resolved Hide resolved
src/core/logger.h Outdated Show resolved Hide resolved
src/core/logger.h Outdated Show resolved Hide resolved
src/core/logger.h Outdated Show resolved Hide resolved
src/core/scriptmodel.cpp Outdated Show resolved Hide resolved
src/core/scriptmodel.cpp Outdated Show resolved Hide resolved
src/core/scriptmodel.cpp Outdated Show resolved Hide resolved
src/utils/log.h Outdated Show resolved Hide resolved
@mgiroday mgiroday force-pushed the use-source_location-for-the-LOG branch from fc65df2 to 9f9193a Compare September 11, 2024 05:51
@mgiroday
Copy link
Contributor Author

Ready for review.

@mgiroday mgiroday merged commit 69ec4a7 into KDAB:main Sep 12, 2024
5 checks passed
@mgiroday mgiroday deleted the use-source_location-for-the-LOG branch September 12, 2024 06:35
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