Skip to content

ci(core-build.yml): add pipeline#8

Open
Helias wants to merge 1 commit into
masterfrom
add-pipeline
Open

ci(core-build.yml): add pipeline#8
Helias wants to merge 1 commit into
masterfrom
add-pipeline

Conversation

@Helias

@Helias Helias commented Aug 29, 2024

Copy link
Copy Markdown
Member

No description provided.

@Helias

Helias commented Aug 29, 2024

Copy link
Copy Markdown
Member Author

build is failing due to some cppcheck:

modules/mod-chat-transmitter/src/Requests/RequestQueryResult.h:30:9: error: There is an unknown macro here somewhere. Configuration is required. If NLOHMANN_JSON_EXPAND is a macro then please configure it. [unknownMacro]
        NLOHMANN_DEFINE_TYPE_INTRUSIVE(QueryResult, queryId, data, columns, success, error, affectedRows)
        ^
modules/mod-chat-transmitter/libs/nlohmann/json.hpp:23135:5: error: syntax error [syntaxError]
    JSON_HEDLEY_WARN_UNUSED_RESULT
    ^
modules/mod-chat-transmitter/libs/nlohmann/json.hpp:5466:5: error: syntax error [syntaxError]
    JSON_HEDLEY_RETURNS_NON_NULL
    ^
modules/mod-chat-transmitter/src/PlayerInfo.h:50:9: error: There is an unknown macro here somewhere. Configuration is required. If NLOHMANN_JSON_EXPAND is a macro then please configure it. [unknownMacro]
        NLOHMANN_DEFINE_TYPE_INTRUSIVE(PlayerInfo, name, guid, level, raceId, classId, gender, accountName, accountGuid, lastIpAddr)
        ^
modules/mod-chat-transmitter/src/Requests/RequestCommandResult.h:22:9: error: There is an unknown macro here somewhere. Configuration is required. If NLOHMANN_JSON_EXPAND is a macro then please configure it. [unknownMacro]
        NLOHMANN_DEFINE_TYPE_INTRUSIVE(CommandResult, commandId, output, success)
        ^
modules/mod-chat-transmitter/src/Requests/RequestElunaError.h:21:9: error: There is an unknown macro here somewhere. Configuration is required. If NLOHMANN_JSON_EXPAND is a macro then please configure it. [unknownMacro]
        NLOHMANN_DEFINE_TYPE_INTRUSIVE(ElunaError, guildId, trace)
        ^

@pangolp

pangolp commented Aug 30, 2024

Copy link
Copy Markdown
Contributor

I really don't know how to fix this, I would be interested in seeing the result, to see if it can be replicated in other modules, and solve these problems in the future, but at this moment, I don't know how to fix it.

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