sync - #2
Open
nanocoh wants to merge 3245 commits into
Open
Conversation
…w-metrics-check test: check flow metrics under bazel
Add README sections for save_display_controls and restore_display_controls, and bring the web_server entry in line with web.tcl: -stop, an OS-assigned default port, and -dir as deprecated rather than required. Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
Both tests were registered against the wrong dependency set, so each failed in one build system while passing in the other. display_state_test imports jsdom, but was registered with JS_FILES, which carries only the product's own JS sources. jsdom comes from :node_modules/jsdom, bundled into DOM_TEST_DATA with the other DOM tests. Node could not resolve the import and the test aborted before running a single case, so nothing it covers was ever actually checked. TestSaveDisplayControls compiles gui/src/stub.cpp to pick up the three gui::Gui symbols web_serve.cpp needs, and stub.cpp's initGui calls DescriptorRegistry::initDescriptors, which lives in init_descriptors.cpp. Bazel's :gui_stub deps on :gui and gets that file; CMake has no equivalent library, since gui_descriptors deliberately holds only the registry and the descriptor implementations live inside the Qt gui target the test cannot link. The reference was therefore unresolvable, and the test now compiles the three Qt-free descriptor sources itself. bufferTreeDescriptor.cpp is not among them despite init_descriptors.cpp calling BufferTree::setSTA: it reaches QColor through colorGenerator.h and does not compile without Qt, and stub.cpp supplies the BufferTree definitions instead. The link failed under GNU ld on every CMake platform, not only the one CI reported; elsewhere a parallel gmake aborted before reaching the target. Bazel is unaffected because lld discards the never-called initGui before its relocation has to resolve. Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
…ebGUI-display-controls-overlays-grids-misc2 Addresses a portion of #10619, 2.5 Display controls — overlays, grids & misc
Signed-off-by: Peter Gadfort <gadfort@zeroasic.com>
pad: remove unused variable
pdn: dont rely on just the intersection for via insertion, check actual layer shape if initial ensure does not yield valid vias
…ction-highlight-inspection master brought in the display-controls/overlays/grids web work, which touched the same files as the selection/highlight/inspection work here. Conflicts resolved in: src/web/src/inspector.js src/web/src/main.js src/web/src/request_handler.cpp src/web/src/request_handler.h src/web/src/style.css src/web/src/ui-utils.js src/web/test/cpp/TestRequestHandler.cpp src/web/test/js/test-ui-utils.js Beyond the textual conflicts: - ScopedDbuFormat takes master's dbDatabase* (multi-die: a hierarchical top chip owns no dbBlock) and keeps this branch's convert_string installation; all call sites updated. - Highlight collection goes through master's clearSelectionHighlights / setSelectionHighlights / setSelectionSetHighlights helpers so highlight_source stays in sync and a "Flywires only" flip re-derives from the right source. appendHighlightShapes also appends the ShapeCollector flight lines this branch added. - writeInspectPayload combines both signatures; use_dbu and the logger are threaded through this branch's writeInspectTrailer helper. - handleOverlayTile keeps master's error handling, coordinate validation, flywire re-derive and highlight_selected gate, plus this branch's stale-selection drop and highlight-group shapes. - applyBounds picks up master's tileSizeCss() and maxUsefulZoom() cap so both the boot and resync paths get them. src/web/test/BUILD merged cleanly but both sides had added a js_test named ui_utils_test; dropped the duplicate. Also drops two no-op apply lambdas in the set_property test descriptor that only discarded a nodiscard any_cast, and switches the marshalling assertions to pointer-form any_cast so a regression reports as a named assertion instead of an escaping bad_any_cast. Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
Do not collect an unrouted net's flywires twice. This branch's ShapeCollector captures drawLine, and for a net with no wire and no guides DbNetDescriptor::highlight draws the driver->sink fan itself, so the descriptor's copy landed on top of the one collectNetFlightLines derives. Keep the derived copy, which honours kMaxFlywires, and drop the descriptor's. Derive highlight-group shapes through appendHighlightShapes instead of calling sel.highlight() directly, so a net parked in a persistent group honours "Flywires only" the way the current selection does, and rebuild the group shapes when the toggle flips. Rebuild highlight geometry after an object moves. A set_property edit only rebuilt the editing session's shapes; every other session was told to redraw, and redrew from its own cache of the old placement. The session's odb callbacks now raise highlight_geometry_stale on inDbPostMoveInst / inDbInstSwapMasterAfter, which fire in all sessions, and the overlay handler re-derives both the selection and the group shapes from it. Re-run a selection-browser refresh that was skipped because a list_selection was already in flight. The outstanding response predates the mutation, so dropping the refresh left the table -- and the row indices inspect/deselect send back -- describing the old set until the next mutation. Each fix has a regression test confirmed to fail without it. Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
…ebGUI-selection-highlight-inspection Addresses a portion of #10619, 2.6 Selection, highlight & inspection
master has landed a large amount of src/web work since this branch was cut,
including its own implementations of things this branch built independently.
Conflict resolutions, by theme:
Both sides added at the same point; kept both:
- ui-utils.js: buildVisibilityFlags/applySelectionFlags next to master's
toast, modal, scale-bar and selection-ownership helpers.
- tile_generator.h: ColoredPolygon next to TileFrame/dbuPrecision.
- request_handler.h: kFind and kContextAction next to kCancel.
- TestRequestHandler.cpp: FindHandlerTest and master's SetPropertyTest
collided at one insertion point; both restored in full.
Same problem solved twice; took master's implementation:
- Selection highlight lines are master's colored FlightLine, not this
branch's pair<Point, Point>; the duplicate ShapeCollector::drawLine
override is gone.
- Highlight groups: master already had highlight_groups plus
rebuildHighlightGroupShapesLocked/removeFromHighlightGroupsLocked, so
handleContextAction's highlight_* branch now drives that state instead of
keeping its own parallel copy. The context menu and the inspector's
highlight button therefore share one code path, and the group index is
validated the way master's handlers validate it.
- web.cpp uses master's assetPathFromTarget(), which strips the fragment as
well as the query, in place of the inline query strip; /download/image
matches on the cleaned path and gets the raw target for its parameters.
- handleOverlayTile is master's rewrite (request validation, error
responses, dpr/tile_px, flywires and geometry-stale rebuilds) with this
branch's colored-polygon channel added back.
Genuinely merged:
- handleSelect carries both the context flag and show_connectivity; the
"keep the inspected object on an empty right-click" rule now also guards
master's runDeselectAction.
- selectAtLatLng keeps this branch's extraction into a reusable function;
master's Ctrl/Cmd+click became a showConnectivity option supplied by the
click handler. lastSelectionBounds now derives from data.bbox rather
than selected[0].bbox, for the reason master gives at that call site: for
a net the two are different rects.
- pulseHighlight/highlightBBox keep the "don't outline a net's full extent"
skip alongside master's animation teardown and multi-selection branch.
Fixes for breakage the merge introduced without conflicting:
- drawColoredPolygons still took (dbu_tile, scale) after master moved every
draw helper to TileFrame. Converted, using toPxXd/toPxYd and
penWidthCss so an oblique edge keeps its slope.
- handleFind called the pre-merge ScopedDbuFormat, writeInspectPayload and
collectMultiHighlightShapes signatures.
- saveImage's split into renderImagePng dropped master's WEB-0024 text, so
renderImagePng now reports the encoded dimensions through optional
out-params and the message stays "Saved {}x{} image to {}".
- highlight_group_polys was missing from two stale-selection cleanups.
Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
Clearing a selection or a highlight from the canvas context menu left the Select/Highlight panel listing objects the server had already dropped. master drives that panel through one hook, app.refreshOverlay (scheduleRefreshOverlay in main.js), which refreshes the overlay tiles and then calls selectionBrowser.scheduleRefresh(). Every other mutation site uses it: the menu bar's "Clear Highlights", the panel's own toolbar, and the selection_invalidated push handler. The context menu predates the hook and ended its request with redrawAllLayers(), which calls the bare refreshOverlay() instead -- so the overlay tiles repainted, which is why the highlight vanished from the canvas, but the panel was never told anything had changed. Route every context action through the shared hooks: - app.refreshOverlay() last, so the panel refresh sees the cleanup. - clear_selections/clear_all also reset the client-side pieces of the selection the server cannot reach -- inspector, selection animation, the outline rectangle and lastSelectionBounds -- mirroring what the selection_invalidated handler does for the same server-side state. - Everything else refreshes the inspector, for the selection count/index and the highlight-group badge. - Only clear_focus_nets/clear_route_guides/clear_all still redraw the base tiles, which are what actually carry focus nets and route guides. The remaining actions live entirely in the overlay and no longer pay for a full base-tile redraw. Also drop the menu's private _toast()/.cm-toast in favour of ui-utils showToast(), which master added for the same purpose: two transient-notice implementations with different placement and timing was one too many. The notice now appears bottom-centre with every other notice, lasts 4s instead of 1.8s, and replaces itself on repeat instead of stacking. Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
… shortcut
Two problems with the Go to Position dialog.
Shift+G opened it with the X field set to "G". The dialog focuses and
select()s its X field synchronously, so the keystroke's own default action
was then delivered to that field and replaced the whole prefilled value with
the shortcut's letter. Ctrl/Cmd+F already called preventDefault for the
browser's find bar; Shift+G needs it for this reason, so the explanation now
sits above both branches.
The dialog also hardcoded microns -- labels, prefill, parsing and the error
message -- while the rest of the UI follows the "Show DBU" setting (Qt's
MainWindow::useDBU). All three fields now read and write whichever unit is
being displayed, the labels say "(µm)" or "(DBU)" to match, and the Size
window is interpreted in the same unit. The prefill picks up a second fix
from formatDbu: it was hardcoded to .toFixed(3), which is one decimal short
of distinguishing adjacent DBU on a 2000 DBU/µm technology.
Doing that needed a showDbu-aware string->DBU parse, which ruler.js already
had as a local copy of the inverse of app.formatDbu. Rather than add a
third copy, the display-unit rules move to ui-utils as pure functions
(formatDbu, parseDbu, formatDistance, unitLabel) taking {showDbu,
dbuPerMicron}; the app object and ruler.js delegate to them. The ruler and
search-nav test harnesses now delegate too, instead of reimplementing
formatDbu/formatDistance as they did before -- a copy in a harness can only
agree with the app by luck.
Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
…lette Find now ADDS to the selection instead of replacing it. Qt's find dialog goes through Gui::select, which hands its matches to MainWindow::addSelected, so successive searches accumulate and a selection the user built by hand survives one; this handler claimed to mirror that flow but overwrote the set. The cycling iterator comes from the insert of the first match, so prev/next still start from what was found. The capture now rasterizes the ruler distance labels. They are L.divIcon <div>s in a marker pane rather than SVG, so the SVG-pane pass could not see them and both Visible and Entire exports kept a ruler's line and ticks while dropping the measurement it exists to show. They are painted from their computed style rather than serialized through an <img> data URL like the SVG panes: a detached document does not inherit the app's stylesheet, so a foreignObject copy would come out with no background and a default font. Two review comments needed no code change: - `Color bg` is already value-initialized, and Color declares `= 0` on every member, so even plain `Color bg;` runs the implicit default constructor and is zero -- not the uninitialized POD the comment describes. - The capture background already guards the transparent cases before falling back to #111. One is declined, with a comment recording why: dark_magenta and purple are both #800080 because gui::Painter::kDarkMagenta and kPurple are defined with identical RGB in gui.h. The swatch has to show what the server will fill with, so "correcting" this one to #8b008b would make the menu lie about the result. The duplication is real, but it has to be fixed in gui::Painter first. FindHandlerTest's findCount() now clears the selection between searches. It asserts pattern matching only, and accumulation means two of the fixture's null-descriptor Selecteds would otherwise be ordered against each other -- Selected::operator< dereferences the descriptor once the payload types match. The new accumulation test pre-seeds a FakeDescriptor-backed entry, whose differing payload type keeps that comparison off the null pointer. Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
Signed-off-by: Peter Gadfort <gadfort@zeroasic.com>
…ebGUI-contextmenu-search-navigation feat(web): add right-click context menu and search/navigation to Web GUI
Conflicts were confined to src/web, where both branches evolved the same overlay/selection code. Resolutions: - generateOverlayTile: both sides appended parameters; combined as (..., visible_layers, dpr, tile_px, colored_polys, labels). - drawTextLabels/renderLabelTile: ported to master's TileFrame API and scaled the font by frame.px_per_css, so labels hold their size on high-DPI displays like the heatmap and pin-label text do. - selectMatchedNets: master improved the fanout-bin body that this branch had factored into the helper; moved that logic into the helper (with a use_dbu parameter) so both bin handlers get it. - handleOverlayTile: kept master's rewrite and grafted the timing-cone and user-label merges onto it. - Message ids: master claims WEB 40-54, so the label messages move to WEB 55-57. - ScopedDbuFormat now takes a dbDatabase*, and main.js gets its bounds and visibility flags from applyBounds()/buildVisibilityFlags(). Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
Grid::checkSetup() validates a -pins layer against a set built from the grid rings and its straps. Every existing case that uses -pins names a strap layer, so the straps_ loop alone satisfies the check and the rings_ loop is uncovered: removing it leaves the pdn regression green. This is core_grid_with_rings_with_straps with -pins metal6 added, the upper layer of its ring, so that case is the control and the two goldens differ only where the option acts: two block terminals with two ports each, and unchanged SPECIALNETS geometry. Confirmed intended behaviour in #11209. Signed-off-by: shivarammysore <shivaram.mysore@gmail.com>
.leaflet-container carries `cursor: default !important`, added after the ruler landed. It ties .ruler-mode-cursor on specificity and wins the source-order tiebreak, so the ruler crosshair had silently stopped appearing; the label tool set an inline style, which a stylesheet !important outranks outright, so it never appeared at all. Qualify both selectors with .leaflet-container to win on specificity regardless of source order, and toggle a class for labels as the ruler does. .leaflet-dragging still outranks both, so panning keeps the grab cursor. Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
add_label is one user-facing command, but the web and Qt implementations
disagreed on what -anchor accepts. Qt's Painter::anchors() spells the nine
anchors with spaces ("top left"); the web labels used underscores and
implemented only five, dropping the four edge-centres. Nothing validated the
string at any layer, so "top left" -- the spelling a Qt user would reach for,
and the only one Painter::stringToAnchor takes -- silently centred the label,
as did any typo.
Adopt the Qt spellings, implement all nine, and refuse a name that is not
among them: WEB-58 from the Tcl path (mirroring Qt's GUI-45, and listing the
choices) and an error response from the WebSocket path. One table now backs
anchorNames(), isValidAnchor() and the placement in drawTextLabels, so a name
cannot be accepted and then not drawn.
The table is duplicated from painter.cpp rather than shared because libweb
has no link dependency on the Qt GUI, so a test pins the spellings instead.
Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
The Inspector already had a select editor for list/bool properties, but it was gated on data.onPropertyEdit. Labels and rulers keep their state client-side and supply onPropertyChange instead, so they always fell through to the free-text contentEditable path: a label's Anchor was a raw string field, and a ruler's Euclidian a raw "true"/"false" one. Typing a name the server does not accept is now an error rather than a silent centring, which makes a text field the wrong control. Let the gate take either callback and report the option as a string through onPropertyChange, matching what the contentEditable path already sends so the existing handlers need no select-specific case. The anchor names come from list_labels rather than a client-side copy, so the picker offers exactly what parseLabelFields will take. The Qt GUI builds the same dropdown from Painter::anchors() (label.cpp). Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
"Size" invites reading it as a distance in the design, which would put it under the um/DBU display-unit switch alongside x and y. It is a font height in pixels and follows neither. Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
… bin computeNetLengthHistogram sized the range with ceil(max_len / bin_width). When max_len is an exact multiple of the snapped width that makes the last bin [max_len - width, max_len), and the longest nets only land in it by clamping. The bins are upper-exclusive and so is select_net_length_bin, so the bar counted nets that double-clicking it then selected none of. floor+1 keeps the last upper edge above max_len and cannot leave an empty trailing bar, since the longest net always falls in it. This is what the fanout histogram already does (max_fanout / bin_width + 1). Also correct two comments about Graph::pinDrvrVertex. They claimed an instance input pin has only a load vertex, so the driver lookup returns null and the cone walk stops one level in. Graph returns the pin's single vertex for every direction except bidirect, where it looks up a separate driver vertex, so the two coincide off bidirect and the fallback is not what carries the walk through input pins. The fallback still earns its place on bidirect pins; only the reasoning was wrong. A test now pins the actual behaviour, and two more characterise the cone over a real STA graph. Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
…ync depths Two independent Qt-parity gaps found in review. save_image drew user labels unconditionally. TileVisibility had no labels field even though the browser already sends draw_labels for overlay tiles, so a saved image could not reproduce a view with Labels turned off. Qt gates RenderThread::drawLabels on the Misc/"Labels" control and its save_image renders through that same path, so it does honour the toggle. Add the field and skip the snapshot when it is clear. "Sync schematic" copied the cone depths across verbatim, but the two views read a depth differently: the timing cone treats 0 as unlimited, while handleSchematicCone expands while d < depth, so 0 means no expansion. The cone panel defaults both depths to 0, so the default full cone synced to a target-only schematic. The event also carried no direction flags, so a direction switched off in the cone still expanded in the schematic. Send the flags and translate in syncCone: disabled becomes 0 and unlimited becomes the control's maximum, which is safe because the server caps at kMaxConeInsts. Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
Release 0.8.19 also moves LLVM from 22.1.8 to 23.1.0. Clang 23 does not compile the bison src/ files that need the gnulib wrapper headers. The Clang-Tidy job stops with "undeclared identifier getenv". Release 0.8.18 keeps LLVM 22.1.8 and has the same live macOS SDK URL. The lock file also gets the correct rules_rust extension digest. A mac build does not evaluate that extension, thus the first commit missed it. Signed-off-by: Vitor Bandeira <vvbandeira@precisioninno.com>
…rade-rules-python-2-3-2 bazel: upgrade rules_python to 2.3.2 and enforce explicit init py
Bump bazel llvm module to 0.8.18 to fix macOS CI
…yer-graph-edges drt: fix init non-pref dir edges for top layer nodes in dr gridgraph
…-tests Signed-off-by: joey <jimmytown999@gmail.com> # Conflicts: # src/cts/test/array.ok # src/cts/test/array_ins_delay.ok # src/cts/test/array_max_wl.defok # src/cts/test/array_max_wl.ok # src/cts/test/array_no_blockages.ok # src/cts/test/array_repair_clock_nets.ok
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
…e limit Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
Only GitHub Actions was watched. Adds pip, npm and docker, and a 7-day cooldown so a bad release is pulled before it reaches a branch. Signed-off-by: Vitor Bandeira <vvbandeira@precisioninno.com>
Signed-off-by: Naveen Venkat <archgen.guest@nyayanidhi.in>
chore(deps): add rate limit and cooldown to dependabot config
Bumps the actions group with 7 updates: | Package | From | To | | --- | --- | --- | | [actions/checkout](https://github.com/actions/checkout) | `6.0.2` | `7.0.1` | | [actions/first-interaction](https://github.com/actions/first-interaction) | `1.3.0` | `3.1.0` | | [actions/download-artifact](https://github.com/actions/download-artifact) | `4.3.0` | `8.0.1` | | [actions/upload-artifact](https://github.com/actions/upload-artifact) | `4.6.2` | `7.0.1` | | [actions/cache/restore](https://github.com/actions/cache) | `5.0.5` | `6.1.0` | | [actions/stale](https://github.com/actions/stale) | `10.3.0` | `11.0.0` | | [actions/setup-java](https://github.com/actions/setup-java) | `5.7.0` | `6.0.0` | Updates `actions/checkout` from 6.0.2 to 7.0.1 - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@de0fac2...3d3c42e) Updates `actions/first-interaction` from 1.3.0 to 3.1.0 - [Release notes](https://github.com/actions/first-interaction/releases) - [Commits](actions/first-interaction@34f15e8...1c46889) Updates `actions/download-artifact` from 4.3.0 to 8.0.1 - [Release notes](https://github.com/actions/download-artifact/releases) - [Commits](actions/download-artifact@d3f86a1...3e5f45b) Updates `actions/upload-artifact` from 4.6.2 to 7.0.1 - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](actions/upload-artifact@ea165f8...043fb46) Updates `actions/cache/restore` from 5.0.5 to 6.1.0 - [Release notes](https://github.com/actions/cache/releases) - [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md) - [Commits](actions/cache@27d5ce7...55cc834) Updates `actions/stale` from 10.3.0 to 11.0.0 - [Release notes](https://github.com/actions/stale/releases) - [Changelog](https://github.com/actions/stale/blob/main/CHANGELOG.md) - [Commits](actions/stale@eb5cf3a...4391f3d) Updates `actions/setup-java` from 5.7.0 to 6.0.0 - [Release notes](https://github.com/actions/setup-java/releases) - [Commits](actions/setup-java@b6effb0...dd06d9c) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: 7.0.1 dependency-type: direct:production update-type: version-update:semver-major dependency-group: actions - dependency-name: actions/first-interaction dependency-version: 3.1.0 dependency-type: direct:production update-type: version-update:semver-major dependency-group: actions - dependency-name: actions/download-artifact dependency-version: 8.0.1 dependency-type: direct:production update-type: version-update:semver-major dependency-group: actions - dependency-name: actions/upload-artifact dependency-version: 7.0.1 dependency-type: direct:production update-type: version-update:semver-major dependency-group: actions - dependency-name: actions/cache/restore dependency-version: 6.1.0 dependency-type: direct:production update-type: version-update:semver-major dependency-group: actions - dependency-name: actions/stale dependency-version: 11.0.0 dependency-type: direct:production update-type: version-update:semver-major dependency-group: actions - dependency-name: actions/setup-java dependency-version: 6.0.0 dependency-type: direct:production update-type: version-update:semver-major dependency-group: actions ... Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
…_actions/actions-c5beab24fd build(deps): bump the actions group with 7 updates
…outline Render polygon floorplan outlines in web tiles
Covers actions, javascript-typescript and python. C and C++ need a build that does not fit a hosted runner. Signed-off-by: Vitor Bandeira <vvbandeira@precisioninno.com>
sequence_style = "block" is not a valid yamlfix enum value; the current yamlfix/pydantic pins reject it, so fmt_yaml_test crashes with a pydantic ValidationError on any un-ignored file. Use "block_style" instead. drop .gemini/config.yaml from yamlfix.ignore It already matches yamlfix.toml's formatting rules. Source: https://github.com/lyz-code/yamlfix/blob/f857ca370af3db7e5e181373d45c06a148fac3f8/docs/index.md?plain=1#L338-L347 Signed-off-by: Vitor Bandeira <vvbandeira@precisioninno.com>
ci: add CodeQL workflow
cts: isolate array tests from detailed placement
Advanced Security is not available when the repo lives under the private org, so the analyze job would fail there. Signed-off-by: Vitor Bandeira <vvbandeira@precisioninno.com>
Commit 39c6cc6 moved actions/first-interaction to v3.1.0 again. v3 uses snake_case input names. The workflow still passes the v1 kebab-case names. Every greet run now fails with "Input required and not supplied: issue_message". v3 also greets a returning contributor as a first-timer when that person has opened PRs but no issue (actions/first-interaction#369). Commit 0226532 reverted the same bump in July for this reason. Pin back to v1.3.0. Ignore major bumps of this action in dependabot, so the same PR does not return after the cooldown. Minor and patch updates on v1 still arrive. Signed-off-by: Vitor Bandeira <vvbandeira@precisioninno.com>
ci: skip CodeQL analysis on private-org repo
ci: revert first-interaction to v1.3.0 and block major bumps
Bumps the actions group with 1 update: [actions/checkout](https://github.com/actions/checkout). Updates `actions/checkout` from 7.0.0 to 7.0.1 - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v7...3d3c42e) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: 7.0.1 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: actions ... Signed-off-by: dependabot[bot] <support@github.com>
…_actions/actions-7a5a078ad4 build(deps): bump actions/checkout from 7.0.0 to 7.0.1 in the actions group
grt: add debug mode for CUGR engine
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
[Describe your changes here]
Type of Change
Impact
[How does this change the tool's behavior?]
Verification
./etc/Build.sh).Related Issues
[Link issues here]